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 no-operation action for key bindings #149

Closed
wants to merge 1 commit into from

Conversation

g-rden
Copy link
Contributor

@g-rden g-rden commented Jul 9, 2024

I didn't want "Key Super_L is not bound" to be displayed when using Super_L do control my WM. I think this could be useful for other people's setups as well.

Signed-off-by: g-rden <[email protected]>
artemsen added a commit that referenced this pull request Jul 9, 2024
Allows to use action_none as key stub.
Adds unit test.

Relates to #149.

Signed-off-by: Artem Senichev <[email protected]>
@artemsen artemsen closed this in 73c5b6f Jul 9, 2024
@artemsen
Copy link
Owner

artemsen commented Jul 9, 2024

In fact we don't need to create one more dummy type, it is enough to add Super to the exception list.
But it is possible now to suppress any key with existing "none" action.

Thank you for PR!

@g-rden
Copy link
Contributor Author

g-rden commented Jul 10, 2024

Awesome, thanks!

@g-rden g-rden deleted the noop branch July 10, 2024 08:53
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