Skip to content

fix(ui): line the sidebar logo up with the page title - #448

Merged
joszz merged 1 commit into
mainfrom
fix/sidebar-brand-alignment
Oct 10, 2026
Merged

joszz merged 1 commit into
mainfrom
fix/sidebar-brand-alignment

Conversation

@joszz

@joszz joszz commented Oct 10, 2026

Copy link
Copy Markdown
Owner

On desktop and tablet the GarageStack logo sat at the very top of the sidebar, about 28px above the page title beside it, so the two headings looked out of line.

  • The sidebar's top padding now follows the main column's padding, and the logo's row is as tall as the page title's line. The logo and every page title now sit at the same height, on every breakpoint.
  • The gap under the logo matches the one under the page header, so the first nav link starts level with the first card.
  • The shared values live in two new tokens, --content-padding and --page-title-height. The phone and tablet breakpoints now set the padding through the token.
  • On phones, the drawer's logo opens at the same height as the top bar's.

Test plan

  • Measured in Playwright on all six views at 1920, 1440, 1024, 820 and 768 px: the logo's middle matches the title's (48 px desktop, 40 px tablet), and the first nav link starts within 2 px of the first card.
  • At 390 px the drawer logo sits at 28 px, against 27.5 px for the top bar's.
  • pnpm exec vitest run (860 tests) and stylelint pass.

🤖 Generated with Claude Code

On desktop and tablet the GarageStack logo sat at the top of the
sidebar, about 28px above the page title next to it. The sidebar now
takes the main column's padding as its top padding (less its 1px top
border), and the brand row takes the title's line height, so the two
share a middle on every page and breakpoint. The gap under the brand
matches the one under the page header, so the first nav link starts
level with the first card.

--content-padding and --page-title-height carry the shared values; the
phone and tablet breakpoints set the padding through the token now. On
phones the drawer's logo opens at the same height as the topbar's.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@joszz
joszz merged commit fd6560d into main Oct 10, 2026
24 checks passed
@joszz
joszz deleted the fix/sidebar-brand-alignment branch October 10, 2026 21:41
@joszz joszz mentioned this pull request Oct 10, 2026
3 tasks done
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant