Opened 10 years ago
Closed 10 years ago
#11808 closed enhancement (no change required)
Implement zoom for About this system
Reported by: | sambuddhabasu1 | Owned by: | jscipione |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | Applications/AboutSystem | Version: | R1/Development |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Platform: | All |
Description
Currently the zoom option to switch between manual fit and best fit is not present for the About this system application. This needs to be implemented.
Attachments (1)
Change History (13)
comment:1 by , 10 years ago
by , 10 years ago
Attachment: | 0001-Implement-zoom-for-About-this-system-application.patch added |
---|
comment:2 by , 10 years ago
patch: | 0 → 1 |
---|
comment:3 by , 10 years ago
Ok great you made the window zoomable, now implement the AboutWindow::Zoom(BPoint where, float width, float height) method so that it zooms helpfully.
comment:4 by , 10 years ago
Hi jscipione, I did not quite understand what you meant by implementing a AboutWindow::Zoom so that it zooms helpfully. It would be really helpful if there were any examples or any existing app which implements these kind of functions. On going through the applications code, I found out that the DeskCalc has a FrameResized function. Is it similar to this? Thanks
comment:6 by , 10 years ago
Resolution: | → invalid |
---|---|
Status: | new → closed |
This makes no sense, zooming this window is useless. Closing as invalid.
comment:8 by , 10 years ago
Generally you want to zoom so that the content fits nicely on the screen and then make the window as tall as comforatable. I don't know an example in the haiku source code but I did some pretty sophisticated zoom code in the paladin editor that takes into account Deskbar's location and size so as to zoom without covering it.
comment:9 by , 10 years ago
Owner: | changed from | to
---|---|
Status: | reopened → assigned |
comment:10 by , 10 years ago
Then the default implementation does what is right: even at full screen size and ridiculously small font the window won't be wide or high enough to fit the whole text.
I still think it is correct to not have a zoom button at all on AboutSystem.
comment:12 by , 10 years ago
Resolution: | → no change required |
---|---|
Status: | assigned → closed |
I am currently working on this bug.