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

Added support for global hotkeys + example #228

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

bibainet
Copy link

@bibainet bibainet commented Sep 28, 2016

Added support for registering and handling global (system wide) hotkeys:

walk.RegisterGlobalHotKey(window, hotkeyID, walk.Shortcut {
    Modifiers: walk.ModControl | walk.ModAlt,
    Key: walk.KeyX,
})
window.Hotkey().Attach(func(hotkeyID int) {
    // Hotkey hotkeyID (Ctrl+Alt+X) pressed somewhere (may be in another application)
})

This functionality depended on changes from my recent pull request in lxn/win

Complete example included.

@bibainet bibainet closed this Mar 24, 2017
@lxn
Copy link
Owner

lxn commented Mar 25, 2017

Hi Alexander,
I'm sorry for not responding to your PR for this long time. Are you still interested in getting this in?

@bibainet
Copy link
Author

Hi Alexander,
I re-opened pull requests, but now there are merge conflicts that need to be resolved.

@bibainet bibainet reopened this Mar 27, 2017
@lxn
Copy link
Owner

lxn commented Mar 27, 2017

Thanks for reopening!

Do you think it would make sense to integrate this somehow with Action?

@bibainet
Copy link
Author

May be. But I do not have much experience with WALK. I will not do this in the near future.

@lxn
Copy link
Owner

lxn commented Mar 28, 2017

No problem. If you don't mind, I will have a look sometime soon.

@bibainet
Copy link
Author

OK. Conflicts resolved.

@bibainet
Copy link
Author

Hi, Alexander.
Should I do something else to complete the merge?

@lxn
Copy link
Owner

lxn commented Apr 1, 2017

Hi Alexander,

Thank you for resolving the conflicts.

The question is if we should put this into Action, where I think it belongs, or leave it separate. I need to think some more about it, but right now there are more pressing issues on my mind, sorry. Hopefully next week.

@kgolding
Copy link

kgolding commented Nov 9, 2018

Hello - can I get an update of this as it's a feature I'm looking for please?

@shivas
Copy link
Contributor

shivas commented Jun 25, 2021

Would love this feature merged too

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.

4 participants