fix(web): align sidebar logo with macOS traffic light buttons#586
fix(web): align sidebar logo with macOS traffic light buttons#586t3dotgg merged 1 commit intopingdotgg:mainfrom
Conversation
Adjust the Electron sidebar header to properly center the T3 wordmark with the native window controls. The logo was sitting ~2px too low and ~8px too close to the traffic lights. - Reduce wordmark container top margin from mt-2 (8px) to mt-1.5 (6px) - Increase Electron header left padding from 82px to 90px for proper traffic light clearance - Match the desktop update button margin to stay aligned Closes pingdotgg#564
|
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)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). 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 |
Summary
Fixes the sidebar logo alignment in the Electron desktop app on macOS. The T3 wordmark was not vertically centered with the native window traffic light buttons — it sat ~2px too low and ~8px too close to the traffic lights.
Closes #564
Changes
mt-2(8px) tomt-1.5(6px), shifting the logo up ~2px to better center with the traffic light buttons.SidebarHeaderleft padding frompl-[82px]topl-[90px], adding 8px of breathing room from the traffic light buttons.mt-2→mt-1.5) so it stays vertically aligned with the wordmark.All three changes are scoped to
apps/web/src/components/Sidebar.tsx. Only Tailwind class adjustments — no logic changes.Before / After
Before


After
Testing
bun lint— 0 errorsbun typecheck— 0 errors