Replies: 1 comment 2 replies
-
I really like this feature, excellent idea. I think currently it makes sense to keep the positional API style for now. Would you be available to pick this up? |
Beta Was this translation helpful? Give feedback.
2 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Problem
Currently,
useRecordHotkeys
records all key presses while active. This creates an accessibility and usability issue:When a user activates hotkey recording via keyboard (e.g., pressing a shortcut to change shortcuts), they are unable to use keys like
Tab
orEnter
to navigate or interact with UI elements such as “Save” or “Cancel” buttons.This behavior breaks the expected keyboard navigation flow and hinders accessibility—especially for users who rely heavily on keyboard input.
Proposed Solution
Add support for excluding specific keys from being recorded while
useRecordHotkeys
is active. This allows essential UI keys (likeTab
orEnter
) to remain functional for navigation and interaction.Suggested API
Using a new options object as parameter:
Alternatively, if you prefer to keep the current positional API style:
Why This Matters
Thank you for considering this improvement! I’d be happy to help test or contribute if this aligns with the project’s goals.
Beta Was this translation helpful? Give feedback.
All reactions