Opened 16 years ago
Closed 16 years ago
#2322 closed bug (fixed)
[Zip-O-Matic] don't need zoom button (easy)
Reported by: | diver | Owned by: | axeld |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | Add-Ons/Tracker | Version: | R1/pre-alpha1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Platform: | All |
Description
Zip-O-Matic don't need zoom button.
Attachments (1)
Change History (9)
comment:1 by , 16 years ago
Summary: | [Zip-O-Matic] don't need zoom button. → [Zip-O-Matic] don't need zoom button (easy) |
---|
comment:2 by , 16 years ago
comment:3 by , 16 years ago
I would assume the patch i just posted will fix this, or it was ok as it was and this ticket should then just be closed as invalid. Note that the zipomatic windows defaults to be wider with this fix than without it.
comment:4 by , 16 years ago
I just looked at the ZipOMaticWindow.cpp - it has special support for zooming and should "fit to size".
comment:6 by , 16 years ago
Just to clarify, I'm talking about Expander. Zip-o-Matic add-on from /boot/beos/system/add-ons/Tracker still has indeed a zoom button. Thanks.
comment:7 by , 16 years ago
just change the code: http://dev.haiku-os.org/browser/haiku/trunk/src/add-ons/tracker/zipomatic/ZipOMaticWindow.cpp
31 32 ZippoWindow::ZippoWindow(BMessage * a_message) 33 : BWindow(BRect(200,200,430,310), "Zip-O-Matic", B_TITLED_WINDOW, B_NOT_V_RESIZABLE), // | B_NOT_ZOOMABLE),
to
31 32 ZippoWindow::ZippoWindow(BMessage * a_message) 33 : BWindow(BRect(200,200,430,310), "Zip-O-Matic", B_TITLED_WINDOW, B_NOT_V_RESIZABLE | B_NOT_ZOOMABLE),
Is it resolved? But I don't compile the code.
comment:8 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Actually, this ticket isn't really valid any more, as pointed out by stippi, Zip-O-Matic's Zoom button does actually work as it's intended to (resize window to fit the length of the filename being expanded), so it's not really necessary or desirable to remove it. Closing this ticket.
http://dev.haiku-os.org/browser/haiku/trunk/src/add-ons/tracker/zipomatic/ZipOMaticWindow.cpp Line 33, this seems to be commented out for some reason.