-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
Control UI: add guarded dashboard keyboard shortcuts #81946
Copy link
Copy link
Open
Labels
P3Low-priority cleanup, docs, polish, ergonomics, or speculative work.Low-priority cleanup, docs, polish, ergonomics, or speculative work.clawsweeper:needs-maintainer-reviewClawSweeper marked this issue as needing maintainer review before automation.ClawSweeper marked this issue as needing maintainer review before automation.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.enhancementNew feature or requestNew feature or requestissue-rating: 🌊 off-meta tidepoolIssue quality rating does not apply to this item.Issue quality rating does not apply to this item.maintainerMaintainer-authored PRMaintainer-authored PR
Description
Metadata
Metadata
Assignees
Labels
P3Low-priority cleanup, docs, polish, ergonomics, or speculative work.Low-priority cleanup, docs, polish, ergonomics, or speculative work.clawsweeper:needs-maintainer-reviewClawSweeper marked this issue as needing maintainer review before automation.ClawSweeper marked this issue as needing maintainer review before automation.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.enhancementNew feature or requestNew feature or requestissue-rating: 🌊 off-meta tidepoolIssue quality rating does not apply to this item.Issue quality rating does not apply to this item.maintainerMaintainer-authored PRMaintainer-authored PR
Type
Fields
Priority
None yet
Split from #11146.
Problem
The Control UI has a command-palette shortcut, but common dashboard chat actions still require mouse interaction. The most obvious gap is the New messages button, which is click-wired only.
Current source evidence:
ui/src/ui/app.tshas a document-level shortcut only forCmd/Ctrl+K/Ctrl+Kto toggle the command palette.ui/src/ui/views/chat.tsrenders New messages with only the button click path toonScrollToBottom.Scope
Add a guarded Control UI shortcut dispatcher for the smallest valuable set of dashboard actions:
/focuses the chat composer when focus is not already inside editable content.Njumps to new messages / bottom when the new-message affordance is present.Escdismisses transient dashboard UI such as command palette, search, menus, notices, or focus state where applicable.Cmd/Ctrl+Kcommand palette behavior.Avoid binding
Spaceas the primary jump shortcut for the first pass because it conflicts with native button/page behavior.Acceptance criteria
Validation expectation
Before merge, run focused UI tests for the touched shortcut/chat path and the repo's changed-file gate in Testbox for user-visible Control UI work.