Fix sidebar v2 thread control overlap#4553
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 6cc9ada. Configure here.
ApprovabilityVerdict: Approved Pure CSS styling changes to fix visual overlap - adds z-index, stacking context, shadows, and background colors. No logic or runtime behavior changes. You can customize Macroscope's approvability policy. Learn more. |
Dismissing prior approval to re-evaluate 0d0996c

What Changed
Updated the sidebar v2 thread lifecycle controls so the Snooze and Settle actions render
over an opaque, row-matched surface with a subtle left-edge fade and shadow.
This prevents long project names from remaining visible underneath the controls when a
thread row is hovered or keyboard-focused.
Why
Long project names could overlap the Snooze and Settle controls, making both the project
name and actions difficult to read.
The existing dark-mode row colors are translucent, so simply reusing those colors did not
fully obscure the text beneath the controls. This change composites the dark row color
against the sidebar background to produce the same appearance with full opacity.
UI Changes
Before
The project name remains visible underneath the lifecycle controls.
After
The project name fades beneath an opaque surface before reaching the Snooze and Settle
controls.
Validation
SidebarV2.tsxChecklist
Note
Low Risk
CSS-only styling in
SidebarV2.tsxwith no logic or API changes.Overview
Fixes long project names bleeding through Snooze/Settle on sidebar v2 card rows when the row is hovered or keyboard-focused.
The absolute action strip now gets stacking (
z-10,isolate), a left gradient fade (beforepseudo-element), a light shadow, and row-matched opaque backgrounds that follow active, selected, or hover state. In dark mode,color-mixcomposites the translucent row tokens against the sidebar so the control area fully covers underlying text without changing settle/snooze behavior.Reviewed by Cursor Bugbot for commit 0d0996c. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Fix action button overlap in SidebarV2 thread rows
Updates the action-controls container in
SidebarV2Rowto prevent thread control buttons from overlapping row content. Adds a gradient backdrop, z-index layering, padding, rounding, and a shadow to the button container, with conditional background colors for active, selected, and hover states (including dark mode viacolor-mix).Macroscope summarized 0d0996c.