Skip to content

make Listener.canonical a staticmethod, this make the call from callabacks easier.#520

Open
paolopas wants to merge 3 commits intomoses-palmer:masterfrom
paolopas:go_static
Open

make Listener.canonical a staticmethod, this make the call from callabacks easier.#520
paolopas wants to merge 3 commits intomoses-palmer:masterfrom
paolopas:go_static

Conversation

@paolopas
Copy link
Copy Markdown

@paolopas paolopas commented Dec 1, 2022

I have added the script tools/keyboard-mixin-test.py as a proof of the concept.
Tested with Linux (xorg).
But I was unable to run the tests, sorry.

this make the call from callabacks easier.
@paolopas
Copy link
Copy Markdown
Author

paolopas commented Dec 2, 2022

One last thing I suggest is to uniform the on_activate callback of HotKey to the same protocol of other events callback (i.e. on_press and on_release).
That is a return False into an on_activate should stop the listener too (currently a StopException is needed).
To achieve this a minor HotKey.press modification should suffice.

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.

1 participant