Skip to content

fix(web-ui): shortcut digit lookup, overrides, editor canvas inheritance#712

Merged
bobleer merged 1 commit into
GCWing:mainfrom
bobleer:fix/web-shortcuts-digit-keys-editor-inheritance
May 14, 2026
Merged

fix(web-ui): shortcut digit lookup, overrides, editor canvas inheritance#712
bobleer merged 1 commit into
GCWing:mainfrom
bobleer:fix/web-shortcuts-digit-keys-editor-inheritance

Conversation

@bobleer
Copy link
Copy Markdown
Collaborator

@bobleer bobleer commented May 14, 2026

Summary

  • Match Digit*/Numpad* codes so Ctrl+digit shortcuts resolve consistently across keyboard layouts.
  • When focus is inside the Monaco editor (editor scope), also evaluate canvas shortcuts so tab-switch and related bindings remain available.
  • Apply user keybinding overrides by replacing modifier flags from stored truthy fields only, avoiding stale catalog modifiers after clears.
  • Persist keybindings as versioned storage and ignore user overrides for scene.openSession (non-customizable).

Verification

  • pnpm run lint:web && pnpm run type-check:web && pnpm --dir src/web-ui run test:run

- Match Digit/Numpad codes so Ctrl+digit bindings work across keyboard layouts
- When focus is editor scope, also evaluate canvas shortcuts (tab switch, etc.)
- Replace shallow merge for user overrides so cleared modifiers do not linger
- Version stored keybindings and skip overrides for scene.openSession
- Add ShortcutManager tests for macOS primary modifier and conflict checks
@bobleer bobleer merged commit 0b51233 into GCWing:main May 14, 2026
4 checks passed
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.

1 participant