Skip to content

Add shortcuts page and block global shortcuts in focused panels#141

Open
SDSLeon wants to merge 3 commits into
masterfrom
lightcode/silver-meadow-04ef4386
Open

Add shortcuts page and block global shortcuts in focused panels#141
SDSLeon wants to merge 3 commits into
masterfrom
lightcode/silver-meadow-04ef4386

Conversation

@SDSLeon

@SDSLeon SDSLeon commented Jun 6, 2026

Copy link
Copy Markdown
Owner
  • Feature + Bugfix: Adds a searchable Shortcuts settings page and prevents global keybindings from firing inside focused UI regions.
  • Add a Shortcuts section in Settings with context tabs and a catalog built from commands, keybindings, and local shortcuts.
  • Introduce composer, panel, and browser focus contexts via data attributes on the thread composer, right panel, and browser surfaces.
  • Tighten when clauses for commands like pane close and thread search so they do not run while those regions have focus.
  • Add browser F5/Shift+F5 reload handling and tests across keybindings, the shortcut catalog, settings routing, and browser panel behavior.

SDSLeon added 3 commits June 3, 2026 02:43
- Add Shortcuts settings section with searchable, context-filtered keybinding list
- Wire shortcuts into settings overlay, sidebar, types, and routing tests
- Extend command when-context with composer, panel, and browser focus flags
- Tag composer, right panel, and browser surfaces with data attributes for focus detection
- Extract shortcut catalog from ShortcutsSettings into shared module
- Add panel, browser, and composer guards to pane.close and thread.search.open
- Resolve panelFocus for overlay surfaces in when context builder
- Cover catalog rows, when clauses, and browser F5 reload in tests
@vercel

vercel Bot commented Jun 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
lightcode-landing Skipped Skipped Jun 6, 2026 5:12pm

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