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

doc: Be clear that add_hotkey is not sufficient #357

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

Conversation

idbrii
Copy link

@idbrii idbrii commented May 29, 2020

Clarify that you need to wait to keep the script active so it can listen
to hotkeys (it does not spawn a process in the background that will wait
for them).

The line "For an example of how to use a callback synchronously, see
wait." indicates that wait is an alternative to add_hotkey and a
helpful way to use it.

wait seems to be a better alternative to while True: pass.

I found it difficult to figure out how to write a script that just waits
for hotkeys, and this addition would have helped me.

Clarify that you need to wait to keep the script active so it can listen
to hotkeys (it does not spawn a process in the background that will wait
for them).

The line "For an example of how to use a callback synchronously, see
`wait`." indicates that `wait` is an alternative to add_hotkey and a
helpful way to use it.

wait seems to be a better alternative to `while True: pass`.

I found it difficult to figure out how to write a script that just waits
for hotkeys, and this addition would have helped me.
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