Improved zoom button behavior

[mmadia]

resize a page horizontally until a horizontal scrollbar disappears, making a window as narrow as possible (with some sanity checks, of course). Vertical resizing could be to full height.

[siarzhuk]

This task was published as GCI 2012 task ​http://google-melange.appspot.com/gci/task/view/google/gci2012/7968215

[axeld]

Thanks for your patch! It goes into the right direction, however, it does not deal gracefully with lines longer than the screen resolution; the StyledEdit window should never be larger than the screen.

Furthermore, I'm not sure if it really makes sense to disable word wrap, and then do the resize -- there are usually complete paragraphs in such documents. In word wrap mode, I think it makes the most sense to keep the current window width, and only resize the height to make it fit the contents. Only in non-word wrap mode, the current method should be applied for the width, but additionally, the height should be changed such that the windows is as large as needed to show all contents.

[ejno1]

Hi axeld, thanks for the feedback.

The window shouldn't become larger than the screen because I clamp it here: min_c(max_c(width, 256.f), fullWidth)

I agree about the word wrap; I didn't think of that. I'll try to implement it when I have access to my other computer in a week or so. I did not resize vertically because of the suggestion in the ticket, but I'll add that as well.

Replying to axeld:

Thanks for your patch! It goes into the right direction, however, it does not deal gracefully with lines longer than the screen resolution; the StyledEdit window should never be larger than the screen.

Furthermore, I'm not sure if it really makes sense to disable word wrap, and then do the resize -- there are usually complete paragraphs in such documents. In word wrap mode, I think it makes the most sense to keep the current window width, and only resize the height to make it fit the contents. Only in non-word wrap mode, the current method should be applied for the width, but additionally, the height should be changed such that the windows is as large as needed to show all contents.

[ejno1]

Improved zoom behavior (modified)

[siarzhuk]

Hi, ejno1! Thank you for your contribution!

I have tried your patch and found that window, that was zoomed cannot be un-zoomed by another click on the same tab icon. You can compare the zoom-unzoom behavior of the patched StyledEdit with the Terminal for example to observe the problem. This functionality is implemented in BWindow::Zoom() and uses the width/height of the BWindow::fPreviousFrame rect to determine if the window should be zoomed or unzoomed. In my opinion it is a good idea to utilize this base functionality and try to adjust the maxWidth and maxHeight by SetZoomLimits(..) call instead of direct overriding Zoom(...) virtual. Could you , please, try to improve your patch in this direction?

PS: BTW, is "ejno1 <none>" the real user info you want to see in Haiku commit log for your commits? I suspect "<none>" e-mail will be catched by git hooks as invalid and prevent us to push it. Could you, please provide a bit more info if possible? Thank you!

[siarzhuk]

Those were taken some years ago as potential GCI tasks. Unfortunately no place is available for them in my schedule at this days.