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

add SuperL/ShiftL ungrabbing #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kataklasm
Copy link

added SuperL and ShiftL ungrabbing to support i3wm keybinds

@feynmanliang
Copy link
Contributor

Could this change potentially cause conflicts with the Shift keybindings (eg https://github.com/gillescastel/inkscape-shortcut-manager/blob/master/normal.py#L70-L78)?

OmerBenHayun added a commit to OmerBenHayun/inkscape-shortcut-manager that referenced this pull request Mar 28, 2023
https://github.com/gillescastel/inkscape-shortcut-manager/pull/26/files
using simonecig PR code
Hi!
Many tiling window managers use by default keybindings with both the Super key (Mod4) and the shift key pressed (i.e. Mod4+Shift+q to close the focused window).
Currently, such key combinations are grabbed and therefore can't be used while the script is running.
I was able to solve this by ungrabbing the Shift_L key with the modifier Mod4Mask (and also Mod1Mask for the Alt key).
This should also resolve gillescastel#6, since keybindings like Shift+t will still
work.
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.

None yet

2 participants