Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When zooming window, the window size range must be adjusted #880

Closed
MatthiasWM opened this issue Jan 4, 2024 · 3 comments
Closed

When zooming window, the window size range must be adjusted #880

MatthiasWM opened this issue Jan 4, 2024 · 3 comments
Assignees
Labels
bug Something isn't working fixed The issue or PR was fixed. Prio 2 - Low documentation error or undocumented side-effect

Comments

@MatthiasWM
Copy link
Contributor

Describe the bug
If a window has a size range (minimum size, maximum size), and the user zooms into the window using Ctrl-Shift-"+", the size range stays in host coordinates, but it should be adjusted to the zoom factor.

To Reproduce
Launch test/tile, resize to minimal size, zoom to 150%. We can make the window even smaller now, causing overlapping.

Expected behavior
The size range should adapt to the zoom factor.

FLTK Version

  • Version: 1.4.0
  • If from Git, branch: master

Operating System / Platform:

  • macOS Sonoma 14.2.1, Xcode 15.1
@Albrecht-S Albrecht-S changed the title When zooming window, the window size range bust be adjusted When zooming window, the window size range must be adjusted Jan 4, 2024
@Albrecht-S
Copy link
Member

the size range stays in host coordinates, but it should be adjusted to the zoom factor.

Good point. To clarify: the size range values stored in Fl_Window must not be changed but the size range values known to the OS or window manager must be modified and communicated to the OS or WM. The latter is obviously platform specific but very likely needed on all four current platforms (counting X11 + Wayland as two distinct platforms).

Issue confirmed on Linux.

@ManoloFLTK
Copy link
Contributor

@MatthiasWM Please, close issue if you confirm it's fixed.

@MatthiasWM
Copy link
Contributor Author

Perfect. Thank you!

@Albrecht-S Albrecht-S added bug Something isn't working fixed The issue or PR was fixed. Prio 2 - Low documentation error or undocumented side-effect labels Jan 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed The issue or PR was fixed. Prio 2 - Low documentation error or undocumented side-effect
Projects
None yet
Development

No branches or pull requests

3 participants