Opened 11 years ago

Closed 10 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:
Has a Patch: no Platform: All

Description

Zip-O-Matic don't need zoom button.

Attachments (1)

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

Download all attachments as: .zip

Change History (9)

comment:1 Changed 11 years ago by diver

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

comment:2 Changed 11 years ago by scottmc

Changed 11 years ago by scottmc

Attachment: zipomatic.diff added

fix to make it not zoomable

comment:3 Changed 11 years ago by scottmc

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 Changed 11 years ago by stippi

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

comment:5 Changed 10 years ago by franxico

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

comment:6 Changed 10 years ago by franxico

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 Changed 10 years ago by chico

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 Changed 10 years ago by anevilyak

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.