#3036 closed bug (fixed)
[DriveSetup] should open main window without bottom scrollbar (easy)
Reported by: | diver | Owned by: | stippi |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | Applications/DriveSetup | Version: | R1/Development |
Keywords: | Cc: | jacob.simpson@…, elikcoreighn@… | |
Blocked By: | Blocking: | ||
Platform: | All |
Description
Just to look more polished.
Attachments (6)
Change History (29)
by , 16 years ago
Attachment: | drivesetup.png added |
---|
comment:1 by , 16 years ago
Summary: | [DriveSetup] should open main window without bottom scrollbar → [DriveSetup] should open main window without bottom scrollbar (easy) |
---|
by , 16 years ago
Attachment: | DriveSetupNew.png added |
---|
comment:2 by , 16 years ago
by , 16 years ago
Attachment: | drivesetup.patch added |
---|
comment:4 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Sorry, didn't notice the patch, but it didn't fix the actual problem. However, I have meanwhile come up with a fix that will always try to open the window such that scrolling is not necessary and also all the columns have their preferred width.
comment:6 by , 15 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Still reproducible in hrev32886, reopening.
comment:7 by , 15 years ago
I've attached screenshot to show current view. Double clicking column separator doesn't change anything.
by , 15 years ago
Attachment: | drivesetup-2.patch added |
---|
Expand window size to make horizonatal scrollbar inactive.
comment:8 by , 15 years ago
Cc: | added |
---|
If I understand what is posted in this ticket, there are two issues.
- The issue that started the ticket seems to be about the horizontal scroll bar being active at the default window sizing when viewing the partition list.
- In certain circumstances, when the Device column is double clicked on the separator, it doesn't resize to display all the text in all the cells of the column without collapsing. Documented in comment:ticket:3036:7
I have attached a patch which I believe addresses issue 1.
comment:10 by , 15 years ago
Version: | R1/pre-alpha1 → R1/Development |
---|
comment:11 by , 15 years ago
There is a patch attached to this ticket. Maybe someone could look at it?
To reproduce this problem one could use:
jam -j2 -q haiku-anyboot-imagege qemu -cdrom haiku/trunk/generated.x86gcc2/haiku-anyboot.image
comment:12 by , 15 years ago
The patch is not a good solution, since obviously BColumnListView::PreferredSize() is expected to return the correct size. The problem simply does not bother me enough so I was too lazy to look into it. :-}
by , 14 years ago
Attachment: | ColumnListView_widthfix.patch added |
---|
Fixes ColumnListView width issues
comment:13 by , 14 years ago
patch: | 0 → 1 |
---|
comment:15 by , 14 years ago
I looked at this a little bit and it seems the issue is caused by TitleView::_VirtualWidth() returning the incorrect width. A patch has been submitted for review.
comment:16 by , 14 years ago
Cc: | added |
---|
comment:17 by , 14 years ago
patch: | 1 → 0 |
---|
comment:18 by , 14 years ago
There's another issue now (or still?) where if the virtual width is larger than the width and scrolled entirely right and the size is then reduced, the scrollbar becomes inaccurate. I'll attempt to work on that next.
comment:19 by , 14 years ago
patch: | 0 → 1 |
---|
comment:20 by , 14 years ago
I see what the other issue that was being referred is and I'll start taking a look at it as well. Also, the rightmost column part can be exceptionally difficult to grab as its right on the edge with the vertical scrollbar, so maybe adding a couple pixels of width wouldn't hurt, but not nearly as much as before.
comment:21 by , 14 years ago
I've noticed that grabbing that rightmost column divider is an issue in Tracker, too. When the window is zoomed to the optimum size in details view mode, there seems to be only a 2 pixel wide area to grab and the divider is actually covered by the scroll bar. Oh, look a ticket: #4318
Also, I think that the DriveSetup should be made zoomable in the same way as Tracker.
comment:23 by , 14 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Fixed for real in hrev38374. When you confirm the fix, make sure to remove the DriveSetup settings file in which it remembers its window position.
comment:24 by , 14 years ago
patch: | 1 → 0 |
---|
increased the window width. check the screen shot and let me know.
thanks, _harsha