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

Bug with 16CF0000 style windows #226

Closed
rp1231 opened this issue Jul 22, 2022 · 4 comments · Fixed by #230
Closed

Bug with 16CF0000 style windows #226

rp1231 opened this issue Jul 22, 2022 · 4 comments · Fixed by #230
Labels
bug Something isn't working Windows BUG Bug comes from Windows

Comments

@rp1231
Copy link

rp1231 commented Jul 22, 2022

When the option is set to "move window" on left mouse button on title bar:
NQZ82zN7yn

There is a bug which affects only 16CF0000 style windows
(eg. notepad++ and Anlink):
AltSnap_p7knPdrZPM

You can only click the top half of the minimize button to minimize the window,
If you click on the bottom half of the minimize window it either glitches or maximizes the window:

lp0vzDjGen.mp4
@RamonUnch
Copy link
Owner

This is a windows bug, there is not much I can do about this outside of disabling titlebar actions all together
See this post #83 (comment) for more details

@rp1231
Copy link
Author

rp1231 commented Jul 22, 2022

Ah ok I see.
In that case,
Would it be possible to add an option in the .ini just to enable a permanent window snapping to the screen borders for all windows in all situations.
That way it doesn't need to detect the titlebar, all windows will just snap to the screen borders.

Window snapping to the screen borders is the reason, I enabled this feature.

Sorry if this doesn't make sense.
I'm just hypothesizing.

@RamonUnch
Copy link
Owner

Actually it is not really possible.
AltSnap only interacts with windows movement for a reason, one of them is Alt+Click, an other is titlebar click.
In this latter case AltSnap detects via a WM_NCHITTEST message that the click was performed in a titlebar and performs the titlebar action instead of letting windows handle the click.
The way to go would be to improve titlebar detection with an additional heuristic in addition to the WM_NCHITTEST message. this could lead to other bugs however. I am still invistigating for a better solution.

@rp1231
Copy link
Author

rp1231 commented Jul 22, 2022

Ah ok I see.
Thanks for the explanation.

@rp1231 rp1231 closed this as completed Jul 22, 2022
@RamonUnch RamonUnch added bug Something isn't working Windows BUG Bug comes from Windows labels Jul 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Windows BUG Bug comes from Windows
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants