Skip to content

fix(web): give sidebar project menu rows the same side padding as other menus - #7913

Open
ishaanko wants to merge 1 commit into
pingdotgg:mainfrom
ishaanko:fix/project-menu-padding
Open

fix(web): give sidebar project menu rows the same side padding as other menus#7913
ishaanko wants to merge 1 commit into
pingdotgg:mainfrom
ishaanko:fix/project-menu-padding

Conversation

@ishaanko

@ishaanko ishaanko commented Aug 22, 2026

Copy link
Copy Markdown

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's px-2 with px-1, and the shared [&_svg]:-mx-0.5 nudge 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.

Before After
before after

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-1 override on sidebar project-scope MenuRadioItems so they keep the shared menu px-2 padding.

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-1 utility class from the 'All projects' and project group MenuRadioItem elements in Sidebar.tsx. This aligns their horizontal padding with the rest of the sidebar menus.

Macroscope summarized fa9ddbd.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 01f3dc8d-235f-4d4b-a42d-4fd3e410c7eb

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Aug 22, 2026
@macroscopeapp

macroscopeapp Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant