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

Key release events seem to be shadowed if a corresponding key press event is defined #159

Closed
epetousis opened this issue Sep 9, 2022 · 3 comments · Fixed by #160
Closed
Assignees
Labels
Bug Something isn't working

Comments

@epetousis
Copy link
Contributor

epetousis commented Sep 9, 2022

Version Information:

  • Distribution Information ( run uname -a ) Linux 5.15.61 #1-NixOS SMP Wed Aug 17 12:24:32 UTC 2022 x86_64 GNU/Linux
  • swhkd version ( swhkd -V ) swhkd 1.2.1

Describe the bug:
I'm trying to set up a simple config for a push-to-talk-style binding. I want to run a command when I press a button, and a command when I release that same button. swhks appears to only be running one of the two key down/up events that are bound.

Expected behavior:
swhks should run both the "activate" command as soon as I press the F18 key, and the "deactivate" command as soon as I release the F18 key.

Actual behavior:
swhks only runs the "activate" command.

To Reproduce:

My config:

f18
    sh ~/discord-ptt.sh activate

@f18
    sh ~/discord-ptt.sh deactivate

Any commands would work, of course - I'm just adding my own for completeness.

Additional information:

I haven't had much of a dig through the code, so it's entirely possible I'm doing something wrong or missed a flag.

@epetousis epetousis added the Bug Something isn't working label Sep 9, 2022
@Shinyzenith
Copy link
Member

Thanks for the thorough report!
@EdenQwQ hi eden could this possibly be a parser issue?

@epetousis
Copy link
Contributor Author

This is actually an issue in swhkd not swkhs, my bad. Looks like the hotkey handler loop exits too early. If I change line 352 in swhkd/daemon.rs to be continue; rather than break;, my release key starts working as expected.

@Shinyzenith
Copy link
Member

Perfect! Feel free to send a pr ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Development

Successfully merging a pull request may close this issue.

4 participants