Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview Updates loading and icon visuals for consistency, including richer sidebar skeleton rows (icon + text), adjusted workflow color swatch sizing/border, and rounded corner tweaks on collapsed sidebar menu triggers. Written by Cursor Bugbot for commit a8edee2. Configure here. |
Greptile SummaryThis PR contains a set of focused, non-breaking UI refinements to the workspace sidebar. The headline change consolidates the separate logo
Confidence Score: 5/5Safe to merge — all changes are visual/UX refinements with no logic regressions. No P0 or P1 issues found. Previous review concerns (two-click UX, stale No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[User clicks logo/collapse area] --> B{isCollapsed?}
B -- Yes --> C[handleExpandSidebar called]
C --> D[e.preventDefault — navigation blocked]
D --> E[toggleCollapsed — sidebar expands]
E --> F[showCollapsedTooltips set to false immediately]
B -- No --> G[Link navigates to /workspace/workspaceId/home]
H[User clicks collapse button] --> I[toggleCollapsed — sidebar collapses]
I --> J[Wait 200ms]
J --> K[showCollapsedTooltips set to true — tooltip enabled]
Reviews (2): Last reviewed commit: "fix(sidebar): remove no-op gap-2 from si..." | Re-trigger Greptile |
...rkspaceId]/w/components/sidebar/components/collapsed-sidebar-menu/collapsed-sidebar-menu.tsx
Show resolved
Hide resolved
|
@greptile |
|
@cursor review |
* 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