feat: centralized shortcut registry + project detail tab shortcuts#96
Closed
NagariaHussain wants to merge 3 commits into
Closed
feat: centralized shortcut registry + project detail tab shortcuts#96NagariaHussain wants to merge 3 commits into
NagariaHussain wants to merge 3 commits into
Conversation
Single source of truth for all keyboard shortcut metadata so the help dialog auto-updates when new shortcuts are added. Also adds O/M/U/R/A hotkeys on the project detail page for quick tab switching. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Changes the "Add assignee" shortcut from A to S to avoid conflict with the new Activity tab shortcut. Adds M→S chord in the task detail sheet to quickly assign the task to yourself. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
NagariaHussain
changed the title
…onflict - Focus scrollable container instead of title input when task sheet opens so S and M→S shortcuts work immediately - Add preventDefault on M chord leader to prevent milestone popover opening - Add shift:false to project page T hotkey so Shift+T only celebrates Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
Author
|
Implemented in 136a61a. All items from the test plan verified:
|
NagariaHussain
added a commit
that referenced
this pull request
Feb 26, 2026
…96 Added shortcut-registry.ts as a single source of truth for all keyboard shortcut metadata. Help dialog (?) now auto-populates from the registry instead of a hardcoded duplicate list. Added useMetaHotkey hook for Cmd/Ctrl combos; useCommandPalette now uses it instead of inline useEffect. Added 5 new tab-switching shortcuts on the project detail page: O (Overview), M (Milestones), U (Updates), R (Requests), A (Activity) — disabled when dialogs/sheets are open. Key decisions: - Registry is a flat array with group field; getShortcutsByGroup() helper preserves insertion order for the help dialog - Tab shortcuts use the enabled option on useHotkey to disable when any dialog/sheet/editing state is active - useMetaHotkey keeps Cmd+K handling clean without inline useEffect Files: shortcut-registry.ts, use-hotkey.ts, CommandPalette.tsx, ShortcutHelpDialog.tsx, ProjectDetailPage.tsx Closes #96 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
shortcut-registry.tsas a single source of truth for all keyboard shortcut metadata (keys, description, group)?) now auto-updates from the registry — no more hardcoded duplicate listuseMetaHotkeyhook for Cmd/Ctrl combos;useCommandPalettenow uses it instead of inlineuseEffectTest plan
?→ help dialog shows all shortcuts from registry (Global, Navigation, Project Detail, Task Detail groups)Cmd+K→ command palette opensyarn buildsucceeds🤖 Generated with Claude Code