Skip to content

feat: add keyboard shortcuts help overlay#173

Draft
vku2018 wants to merge 1 commit into
StableRoute-Org:mainfrom
vku2018:grantfox-148-keyboard-shortcuts
Draft

feat: add keyboard shortcuts help overlay#173
vku2018 wants to merge 1 commit into
StableRoute-Org:mainfrom
vku2018:grantfox-148-keyboard-shortcuts

Conversation

@vku2018

@vku2018 vku2018 commented Jul 4, 2026

Copy link
Copy Markdown

Summary

  • add a shared ShortcutsHelp client component that opens from ? / Shift+/ and ignores editable fields
  • render the overlay as a named modal dialog with Escape, close-button, and backdrop dismissal
  • mount the overlay once from src/app/layout.tsx so it is available on every route
  • document the shortcut in the README

Closes #148.

Validation

  • PASS: npm test -- ShortcutsHelp --runInBand (7 tests)
  • PASS: npm run lint
  • PASS: git diff --cached --check
  • BASELINE FAIL: npm run build fails on src/app/quote/Client.tsx:41 with Cannot find name 'assetsDiffer'
  • BASELINE FAIL: npm test -- --runInBand fails only in src/app/quote/page.test.tsx with ReferenceError: assetsDiffer is not defined; 22 suites and 111 tests pass

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.

Add a keyboard-shortcuts help overlay opened with the ? key

2 participants