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

issue with resizing views #2460

Open
killown opened this issue Sep 1, 2024 · 1 comment
Open

issue with resizing views #2460

killown opened this issue Sep 1, 2024 · 1 comment
Labels

Comments

@killown
Copy link
Contributor

killown commented Sep 1, 2024

if you keep resizing a view until it disappear, wayfire events will report a weird data like {'x': 1892042110, 'y': 29637}

{'event': 'view-workspace-changed', 'from': {'x': 1892042110, 'y': 29637}, 'to': {'x': 1, 'y': 0}, 'view': {'activated': True, 'app-id': 'kitty', 'base-geometry': {'height': 6, 'width': 1, 'x': 4, 'y': 1070}, 'bbox': {'height': 15, 'width': 14, 'x': 0, 'y': 1065}, 'focusable': True, 'fullscreen': False, 'geometry': {'height': 14, 'width': 9, 'x': 0, 'y': 1066}, 'id': 314, 'last-focus-timestamp': 10820901004393, 'layer': 'workspace', 'mapped': True, 'max-size': {'height': 0, 'width': 0}, 'min-size': {'height': 0, 'width': 0}, 'minimized': False, 'output-id': 1, 'output-name': 'DP-1', 'parent': -1, 'pid': 147176, 'role': 'toplevel', 'sticky': False, 'tiled-edges': 0, 'title': '~', 'type': 'toplevel', 'wset-index': 1}}

I believe the issue is that Wayfire does not enforce minimum width/height limits for resizing views.
From a logical standpoint, allowing a view to be resized to a point where it becomes impossible to focus on it is not meaningful.

@killown killown added the bug label Sep 1, 2024
@soreau
Copy link
Member

soreau commented Sep 1, 2024

Probably could use PR #2434 and add a minimum size of 1x1 or so, for views that don't set a min size.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants