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)

zipomatic.diff (645 bytes ) - added by scottmc 16 years ago.
fix to make it not zoomable

Download all attachments as: .zip

Change History (9)

comment:1 by diver, 16 years ago

Summary: [Zip-O-Matic] don't need zoom button.[Zip-O-Matic] don't need zoom button (easy)

by scottmc, 16 years ago

Attachment: zipomatic.diff added

fix to make it not zoomable

comment:3 by scottmc, 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 stippi, 16 years ago

I just looked at the ZipOMaticWindow.cpp - it has special support for zooming and should "fit to size".

comment:5 by franxico, 16 years ago

Zoom button isn't there anymore. Please close this ticket.

comment:6 by franxico, 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 chico, 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 anevilyak, 16 years ago

Resolution: fixed
Status: newclosed

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.

Note: See TracTickets for help on using tickets.