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

New Option: Prevent window snapping with Super key. #79

Merged

Conversation

asteinba
Copy link
Contributor

@asteinba asteinba commented Nov 11, 2023

Hi,

I just started using gSnap, wonderful extension! This PR adds a new option that allows to prevent window snapping by holding down the Super key.

Motivation:

I really like the automatic snapping (so without the need to press CTRL) but from time to time I want to loosely position a window. I added this option so that you can hold Super to disable the window snapping (so basically the opposite from the CTRL option).

Also I'm using the Super key to move (with left click) and resize (with right click) windows but this doesn't work with automatic snapping.

I also think the Super key is the perfect candidate for this as you already use Alt and CTRL for other options.

I hope this feature is fine. Let me know if I need to do any further adjustments :).

Edit: Ah and I just noticed the isHolding function was not working for the SUPER modifier, so I fixed that as well :).

@asteinba asteinba changed the title New Option: Prevent snapping with Super key. New Option: Prevent window snapping with Super key. Nov 11, 2023
@asteinba asteinba force-pushed the feature/prevent-snapping-with-super branch from 29f1e45 to f4413b4 Compare November 11, 2023 02:08
Copy link
Member

@meronz meronz left a comment

Choose a reason for hiding this comment

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

lgtm;

I need to test the changes and see if there are some regressions, but all looks good. I'll test it as soon as possible :)

@asteinba
Copy link
Contributor Author

asteinba commented May 6, 2024

Hey @meronz,

I'm using this since the last 6 months and didn't notice any issues. Can we merge this?

@meronz meronz merged commit bb67a12 into GnomeSnapExtensions:main May 18, 2024
1 check passed
@meronz
Copy link
Member

meronz commented May 18, 2024

Sorry for the delay. I merged the PR and I'll do some testing and push the release to EGO today.

@meronz meronz mentioned this pull request May 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants