improvement(sidebar): expand sidebar by hovering and clicking the edge#3830
improvement(sidebar): expand sidebar by hovering and clicking the edge#3830waleedlatif1 merged 6 commits intostagingfrom
Conversation
PR SummaryMedium Risk Overview Adds new global commands/shortcuts for creating a workflow ( Adjusts the universal search modal horizontal positioning to account for the current Written by Cursor Bugbot for commit ba03206. Configure here. |
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Greptile SummaryThis PR improves sidebar UX by adding an invisible 8 px hit zone on the sidebar's right edge that allows users to expand (when collapsed) or resize (when expanded on a workflow page) without relying solely on the top button. It also corrects the Confidence Score: 5/5Safe to merge — no P0 or P1 issues found; all changes are well-reasoned and structurally correct. The structural overflow-hidden fix is sound, the --sidebar-width CSS variable is reliably maintained throughout the codebase, the search-modal centering formula is mathematically verified, the ARIA/keyboard concerns from the prior review thread are fully addressed, and the codeToKey international-keyboard fallback is correct. No regressions identified. No files require special attention. Important Files Changed
Reviews (4): Last reviewed commit: "fix(sidebar): guard add-workflow shortcu..." | Re-trigger Greptile |
…enter search modal, fix edge ARIA
|
@cursor review |
|
@greptile |
…ift+P to avoid browser close-window conflict
|
@greptile |
|
@cursor review |
|
@greptile |
|
@cursor review |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
#3830) * improvement(sidebar): expand sidebar by hovering and clicking the edge * improvement(sidebar): add keyboard shortcuts for new workflow/task, center search modal, fix edge ARIA * improvement(sidebar): use Tooltip.Shortcut for inline shortcut display * fix(sidebar): change new workflow shortcut from Mod+Shift+W to Mod+Shift+P to avoid browser close-window conflict * fix(hotkeys): fall back to event.code for international keyboard layout compatibility * fix(sidebar): guard add-workflow shortcut with canEdit and isCreatingWorkflow checks
* feat(logs): add additional metadata for workflow execution logs * Revert "Feat(logs) upgrade mothership chat messages to error (#3772)" This reverts commit 9d1b976. * Fix lint, address greptile comments * improvement(sidebar): expand sidebar by hovering and clicking the edge (#3830) * improvement(sidebar): expand sidebar by hovering and clicking the edge * improvement(sidebar): add keyboard shortcuts for new workflow/task, center search modal, fix edge ARIA * improvement(sidebar): use Tooltip.Shortcut for inline shortcut display * fix(sidebar): change new workflow shortcut from Mod+Shift+W to Mod+Shift+P to avoid browser close-window conflict * fix(hotkeys): fall back to event.code for international keyboard layout compatibility * fix(sidebar): guard add-workflow shortcut with canEdit and isCreatingWorkflow checks * feat(ui): handle image paste (#3826) * feat(ui): handle image paste * Fix lint * Fix type error --------- Co-authored-by: Theodore Li <theo@sim.ai> * feat(files): interactive markdown checkbox toggling in preview (#3829) * feat(files): interactive markdown checkbox toggling in preview * fix(files): handle ordered-list checkboxes and fix index drift * lint * fix(files): remove counter offset that prevented checkbox toggling * fix(files): apply task-list styling to ordered lists too * fix(files): render single pass when interactive to avoid index drift * fix(files): move useMemo above conditional return to fix Rules of Hooks * fix(files): pass content directly to preview when not streaming to avoid stale frame * improvement(home): position @ mention popup at caret and fix icon consistency (#3831) * improvement(home): position @ mention popup at caret and fix icon consistency * fix(home): pin mirror div to document origin and guard button anchor * chore(auth): restore hybrid.ts to staging * improvement(ui): sidebar (#3832) * Fix logger tests * Add metadata to mothership logs --------- Co-authored-by: Theodore Li <theo@sim.ai> Co-authored-by: Waleed <walif6@gmail.com> Co-authored-by: Theodore Li <theo@sim.ai>

Summary
Type of Change
Testing
Tested manually
Checklist