#8670: Improved zoom button behavior ---------------------------------------+---------------------------- Reporter: mmadia | Owner: siarzhuk Type: enhancement | Status: assigned Priority: normal | Milestone: R1 Component: Applications/StyledEdit | Version: R1/Development Resolution: | Keywords: Blocked By: | Blocking: 8665 Has a Patch: 1 | Platform: All ---------------------------------------+---------------------------- Comment (by 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 [comment:4 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. -- Ticket URL: <http://dev.haiku-os.org/ticket/8670#comment:5> Haiku <http://dev.haiku-os.org> Haiku - the operating system.