-
-
Notifications
You must be signed in to change notification settings - Fork 669
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
window resizable and minimizable set to False ignored #1608
Comments
Thanks for the report. To clarify the report - the window has resizing disabled if you grab the window corners; however, the maximize button on the toolbar hasn't been disabled. The |
@freakboy3742 "To clarify the report - the window has resizing disabled if you grab the window corners; " Actually, on my system, the window can be made larger, but not smaller, when you grab any of the window edges, corners, sides, top, bottom. |
@annetracy Interesting - sounds like we've got some odd interactions of the window manager at play here. The "bigger but not smaller" resize behavior has been reported in the past (#1205); AFAICT, the cause is unrelated to the issue of being able to resize at all. Disabling all resizing almost certainly requires a window-level feature to be disabled; the "allow getting smaller" is a bug in how and when the layout algorithm is evaluating window layout. |
OK here's the thing. in toga_gtk.window.init(... I added the following line.
the window.init code now reads ` def init(self, interface, title, position, size):
The window now functions as I expected... see in the bug report. |
Yup - that's not surprising at all. Most of the time, when there's something fundamental like "resize" not being honoured, it's literally a case of an API endpoint not being hooked up. Care to try your hand at turning this into a PR? |
I would if I knew what a PR was |
Ah! Pull Request |
I hope I've done that right. I've edited the file using the create fork method in the browser. |
@annetracy In the case of this feature, the in-browser editor is sufficient, because the fix only requires a change to a single file. If you're planning to make a larger contribution (e.g., something that requires editing more than one file), you'll need to become familiar with Github's contribution workflow. |
Fixed by #1609. |
Thanks... in the meantime I'm working my way through understanding the whole clone to local, Pull commit to my remote, thing. It way, way different from what I was used to in my working life. |
It's definitely a different way of looking at development, and there's a learning curve. All I can say is that the effort is worth it. Pretty much all Open Source development uses some variation on that theme. Even if you're not contributing to Open Source projects, having good version control and pull request handling can radically change how you view development of your own private projects. |
Describe the bug
When resizable is set to False, the setting is ignored
` #############################################################
## Load the Main Window
`
To Reproduce
Create a Toga window using the above code.
Expected behavior
The resize button on the Window Decoration should not be available. Thereby making it impossible to select it.
Screenshots
Environment:
Operating System: Manjaro Linux
KDE Plasma Version: 5.25.5
KDE Frameworks Version: 5.98.0
Qt Version: 5.15.6
Kernel Version: 5.15.71-1-MANJARO (64-bit)
Graphics Platform: X11
GTK2 V 2.24.33
GTK3 V 3.24.34
Python 3.10.7
toga_core-0.3.0.dev38
toga_gtk-0.3.0.dev38
The text was updated successfully, but these errors were encountered: