Skip to content

fix: match the header logo size to the console - #649

Merged
AbhishekDoshi26 merged 2 commits into
mainfrom
mc/match-logo-size-to-console
Sep 5, 2026
Merged

fix: match the header logo size to the console#649
AbhishekDoshi26 merged 2 commits into
mainfrom
mc/match-logo-size-to-console

Conversation

@easymac

@easymac easymac commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Status

READY

Description

The docs header logo rendered much larger than the same wordmark in the console. Starlight's SiteTitle.astro sizes the logo to fill the nav bar (height: calc(var(--sl-nav-height) - 2 * var(--sl-nav-pad-y))), which works out to 40px tall on desktop and 32px below the 50em breakpoint. The console renders the identical lockup at 22px ([&>svg]:h-5.5 in app/components/nav/app-sidebar.tsx).

This pins .site-title img to 1.5rem (24px) — a hair above the console's 22px, which read as slightly too small in the docs header.

Both SVGs share viewBox="0 0 642 140", so setting the height alone gives the matching width for free: 183.4px to 110.1px.

Worth knowing:

  • The size is now fixed at every viewport. It previously grew from 32px to 40px at the 50em breakpoint, because --sl-nav-height changes there. The console has no responsive variants, so neither does this.
  • Only height is overridden. Starlight's width: auto and max-width: 100% still apply, so the aspect ratio and the narrow-viewport clamp are unchanged.
  • The rule is unlayered, so it beats Starlight's @layer starlight.core without !important. Moving it inside a layer would break that.

Verified in the dev server in both light and dark themes.

@AbhishekDoshi26
AbhishekDoshi26 merged commit f2f7061 into main Sep 5, 2026
5 checks passed
@AbhishekDoshi26
AbhishekDoshi26 deleted the mc/match-logo-size-to-console branch September 5, 2026 18:01
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.

2 participants