fix(web): give sidebar project menu rows the same side padding as other menus - #7913
fix(web): give sidebar project menu rows the same side padding as other menus#7913ishaanko wants to merge 1 commit into
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
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:
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 |
ApprovabilityVerdict: Skipped Macroscope did not run approvability analysis for this PR. Macroscope could not determine whether this PR modifies its approvability configuration, so the PR was not approved automatically. A PR that may change the rules that govern approval is never approved automatically. |
The project filter dropdown in the sidebar felt cramped: the folder icon sat 2px from the left edge of the hover highlight.
The two
MenuRadioItems in the project scope menu overrode the shared menu item'spx-2withpx-1, and the shared[&_svg]:-mx-0.5nudge pushed the icon to the edge. Dropping the override gives the rows the same 8px side padding as every other menu, and the folder icon now lines up with the trigger's icon.Made with Claude Fable 5 in Claude Code.
Note
Low Risk
Cosmetic className-only padding tweak on a dropdown; no logic, data, or security impact.
Overview
Removes the
px-1override on sidebar project-scopeMenuRadioItems so they keep the shared menupx-2padding.That extra 4px of inset keeps the folder/favicon from sitting on the hover highlight edge and lines the icons up with the filter trigger.
Reviewed by Cursor Bugbot for commit fa9ddbd. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Fix sidebar project menu row side padding to match other menus
Removes the
px-1utility class from the 'All projects' and project groupMenuRadioItemelements in Sidebar.tsx. This aligns their horizontal padding with the rest of the sidebar menus.Macroscope summarized fa9ddbd.