-
Notifications
You must be signed in to change notification settings - Fork 117
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
[BUG] Scoped hotkeys still trigger when scope is not active #1058
Comments
After a little bit of experimentation I think this might be a case of scope behaviour being a little unintuitive. When setting the I think what's happening is that even though the wildcard scope From this perspective I can sort of see why my scoped hotkey was running when the wildcard scope was active (no I guess I was under the impression that the wildcard scope behaved just like every other scope, and therefore when it wasn't active, hotkeys that were unscoped would also be disabled. I suppose then, the way to achieve the behaviour I'm after would be to ensure that none of my hotkeys are in the wildcard scope (unless I truly want them available at all times), and to ensure that I set the tl;dr:
|
Yes this behavior will change in version 5. |
@JohannesKlauss I have just run into this behaviour myself. My use case is probably a pretty common one:
Thus far I have been unable to get this working in an easy way. I think the easiest "API" for users is to simply rely on nesting for specificity. That is, the highest-level Alternatively, you could literally just expose a Any estimate for v5 landing, and what are the proposed changes? |
A prerelease is already published under 5.0.0-1 I don't know if and when I'll be able to finish it, since I just don't have time for this project anymore. |
Thanks, I see that release here: https://github.com/JohannesKlauss/react-hotkeys-hook/releases/tag/v5.0.0-1 I see the change to How could we easily "override" global scope hotkeys when some condition is met? It seems to me users will have to maintain global state outside of Edit: In other words, from what I can tell, there is no declarative way to enable or disable scopes. It only works through imperative function calls like In regards to maintenance – this package is used by 23k projects at least. I myself am a very happy (albeit very recent) user. I am sure we could make a successful call for sponsorship, or else find other potential maintainers willing to step up. You have a great thing here! |
Describe the bug
When creating a hotkey and setting the
scopes
option to restrict the hotkey to a specific scope, the hotkey triggers regardless of what the active scope is.In fact, I'm not even sure the scope is being set against the hotkey as when the context's hotkeys collection is dumped there isn't any scopes value against it.
To Reproduce
Repro codesandbox here: https://codesandbox.io/s/react-hotkeys-hook-scopes-not-working-yny6kl?file=/src/App.tsx
Expected behavior
I would expect the scoped hotkey to only trigger when one of the scopes it is set to use is active
Screenshots
N/A
Desktop (please complete the following information):
N/A - happens across windows & osx in chrome & safari
The text was updated successfully, but these errors were encountered: