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

Prevent hotkey when dialog open #9

Merged
merged 3 commits into from
Dec 2, 2023

Conversation

PhilETaylor
Copy link
Contributor

Closes #3

Prevent hotkey invoking when the dialog is open

(Need to test this once the build is previewable, if it works then it can be merged, and tagged if possible please)

Copy link

vercel bot commented Dec 2, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
cmd-dialog ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 2, 2023 3:58pm

@PhilETaylor
Copy link
Contributor Author

nope its broke haha - ill try again

@PhilETaylor
Copy link
Contributor Author

ok using the direct DOM approach works perfectly. This can be tested now

When the dialog is shown, the hotkeys do not work, when the dialog is not shown the hotkeys work

This fixes #3 which was a bug that when you type s in the filter of the dialog and you had a hotkey of s the hotkey would fire, which would be wrong if you were trying to type sea for example.

@OzzyCzech OzzyCzech merged commit 9a85e59 into OzzyCzech:main Dec 2, 2023
14 checks passed
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.

Hotkeys integration bug,
2 participants