Ticket #2322 (closed bug: fixed)

Opened 2 years ago

Last modified 17 months ago

[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: Has a Patch: no
Platform: All Blocking:

Description

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

Attachments

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

Change History

comment:1 Changed 2 years ago by diver

  • Summary changed from [Zip-O-Matic] don't need zoom button. to [Zip-O-Matic] don't need zoom button (easy)

comment:2 Changed 2 years ago by scottmc

Changed 2 years ago by scottmc

fix to make it not zoomable

comment:3 Changed 2 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 2 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 17 months ago by franxico

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

comment:6 Changed 17 months 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 17 months 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 17 months ago by anevilyak

  • Status changed from new to closed
  • Resolution set to fixed

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.