fix(web): center logo in sidebar header - #2693
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 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 |
ApprovabilityVerdict: Approved This is a CSS-only change adding No code changes detected at You can customize Macroscope's approvability policy. Learn more. |
Add `justify-center` to the logo link so the T3 wordmark and "Code" label are horizontally centered within the full-width flex-1 link, regardless of sidebar width. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Note 🤖 GPT-5.6 Sol responding on behalf of Theo Closing this PR after an automated pass over open pull requests. Old sidebar alignment change targets an interface that has since moved. Alignment was fixed in #6086. |
What Changed
Added
justify-centerto the logo link inSidebarChromeHeader.One class. One line changed.
Why
I use T3 Code daily and the logo being left-aligned in the sidebar header has been bugging me for a while. It looks off — especially
when you resize the sidebar and the wordmark just sits there hugging the left edge instead of sitting naturally in the center like
every other app does.
The logo link already has
flex-1on it, meaning it spans the full width of the header. The fix is literally just addingjustify-centerso the T3 wordmark and "Code" label are centered within that space. No layout changes, no new elements, no sideeffects — the sibling
SidebarTriggeris outside the link and unaffected.UI Changes
Before
After
Checklist
Note
Low Risk
Single CSS utility on a sidebar header link; no logic, routing, or data changes.
Overview
Sidebar header branding is horizontally centered in
SidebarChromeHeader: the homeLinkthat shows the T3 wordmark, "Code" label, and stage badge now includesjustify-centeralongside its existingflex flex-1layout, so the cluster sits in the middle of the header instead of left-aligned.No other layout or behavior changes; the mobile
SidebarTriggerstays outside that link.Reviewed by Cursor Bugbot for commit 41cd0ba. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Center logo in sidebar header
Adds
justify-centerto the flex container in theSidebarChromeHeaderwordmark link in Sidebar.tsx, horizontally centering the logo within the header.Macroscope summarized 41cd0ba.