Opened 16 years ago
Closed 16 years ago
#3767 closed enhancement (fixed)
[PATCH] enhancements to DiskUsage
| Reported by: | stpere | Owned by: | axeld |
|---|---|---|---|
| Priority: | normal | Milestone: | R1 |
| Component: | Applications | Version: | R1/pre-alpha1 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: | ||
| Platform: | All |
Description
As said in others ticket (I hope this won't be considered as duplicates, sorry if it is) there are some usability issues with DiskUsage. My patch fixes two of those :
- "Jigger" when resizing the window
- Zoom button not working (in appearance, I know it does something, but it's not obvious)
So, basically, I just removed the proportion constraint on the window, which was working weird, and placed it on the pieView (to avoid having it clipped).
Resizing is now smooth and the Zoom button zooms to the largest square window displayable, which I think is what it should do.
I might tackle a few of the todos in that application.
Attachments (3)
Change History (6)
by , 16 years ago
| Attachment: | patch_diskusage added |
|---|
by , 16 years ago
| Attachment: | patch_diskusage_2 added |
|---|
comment:1 by , 16 years ago
The second version of the patch further enhance DiskUsage so that clicking "Get Info" calls Tracker's GetInfo..
I didn't remove the old InfoWindow as it's used if the messenger can't find tracker. I wasn't sure if it was necessary to keep that old version (my intuition tells me it's cluttering the sources), but I kept it.
TODO : The old InfoWindow was positioned according to where you clicked. I didn't find a way with Tracker's GetInfo to get the same result.. Maybe I could add attributes to the BMessage passed to include the BPoint and modify tracker accordingly.
by , 16 years ago
| Attachment: | patch_diskusage_3 added |
|---|
comment:2 by , 16 years ago
The third version of the patch also fixes drawing artefacts when moving around the info window. The issue is that there is a check in Draw() to not redraw if the updated area isn't intersecting with the pieView area.. problem is that we did an inset on that rect right before that check.. so if we had to redraw something in the margins of the pieview, it was skipped, leaving artefacts..
I just displaced the inset after the intersection check.
Additional enhancements