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

fix: float windows by default if they have WS_POPUP style #941

Merged
merged 5 commits into from
Jan 29, 2025

Conversation

ChichiMsdk
Copy link
Contributor

@ChichiMsdk ChichiMsdk commented Jan 25, 2025

This is to set windows meant to have a specific size (but they are still resizable) to be in floating mode instead of tiling by default.

This could maybe replace the workarounds using the window class for some explorer's popups. Would have to check if those windows have the 'WS_POPUP' flag or not.

Without this, many windows are improperly resized and makes the tiling manager almost unusable, especially with softwares like Visual Studio or Ghidra.

@lars-berger
Copy link
Member

Good point! Think this would make a lot of sense to add as a default 👌

Thank you for the PR

Copy link
Contributor Author

@ChichiMsdk ChichiMsdk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure if this is the correct syntax for an "or" condition in rust as I've never programmed in it. Hoping it is correct

@lars-berger lars-berger changed the title bugfix: Added "is_popup" function to set to floating windows having the WS_POPUP style flag fix: float windows by default if they have WS_POPUP style Jan 29, 2025
@lars-berger lars-berger merged commit d3186ac into glzr-io:main Jan 29, 2025
1 of 2 checks passed
@lars-berger
Copy link
Member

Awesome 🙏 Made a quick linting fix (hope it's ok I pushed directly to the branch).

I held off including this in the current release to give some more time for testing with different apps. From what I've seen in some quick initial testing though, it's looking good!

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

Successfully merging this pull request may close these issues.

2 participants