feat(sidebar): add Cmd+B shortcut to toggle sidebar collapse#5618
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview
Reviewed by Cursor Bugbot for commit aa8f7b8. Configure here. |
Greptile SummaryThis PR adds a keyboard shortcut for collapsing and expanding the workspace sidebar. The main changes are:
Confidence Score: 5/5This looks safe to merge.
Important Files Changed
Reviews (4): Last reviewed commit: "fix(sidebar): don't dead-zone Cmd+B on r..." | Re-trigger Greptile |
|
@cursor review |
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit aa8f7b8. Configure here.
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit aa8f7b8. Configure here.
Summary
Mod+B(⌘B / Ctrl+B) as a global command to toggle the workspace sidebar collapsed/expandedCOMMAND_DEFINITIONSregistry and the sidebar's existinguseRegisterGlobalCommandsblock, calling the sametoggleCollapsedstore action as the collapse buttonMod+Bvia its existingdata-owned-shortcutsmechanism so bold keeps working inside the editor (same pattern it already uses forMod+Klinks)SidebarTooltipgains an optionalshortcutprop renderingTooltip.Shortcut)Type of Change
Testing
Typecheck clean (one pre-existing unrelated error on staging), global-commands-provider and editor keymap tests pass (30/30)
Checklist