Skip to content

Fix navbar logo offset on wide screens#27

Open
maddy-abrahams wants to merge 5 commits into
mainfrom
fix/navbar-logo-full-bleed-header
Open

Fix navbar logo offset on wide screens#27
maddy-abrahams wants to merge 5 commits into
mainfrom
fix/navbar-logo-full-bleed-header

Conversation

@maddy-abrahams

@maddy-abrahams maddy-abrahams commented Jun 18, 2026

Copy link
Copy Markdown

Summary

  • Decouples SiteHeader's logo/nav row from the page content's max-width container. Previously the header centered inside the same max-width box as the content below it, so on wide monitors the logo sat hundreds of pixels from the left edge instead of hugging it.
  • SiteHeader now uses fixed padding only (px-4 sm:px-6 lg:px-8), so the logo stays a constant distance from the viewport edge at any screen width.
  • Removed the now-unused contentMaxWidth prop from SiteHeader and updated all callers (standard layout, analysis layout, experiments/ndeltas page, experiments/ndeltas/[modelId] page).
  • Applied the same fix to the /about page's own header: dropped its max-width container so the logo hugs the left edge, matching the rest of the site.

Test plan

  • Load / and resize the browser from mobile width up to an ultra-wide monitor width -- confirm the logo stays anchored near the left edge throughout, with no large gap appearing on wide screens.
  • Check /analysis/[configId], /experiments/ndeltas, and /experiments/ndeltas/[modelId] -- header should still render correctly (no longer width-matched to that page's wider content grid, which is expected.
  • Check /about -- header logo should sit at the same left-edge position as the home header.

🤖 Generated with Claude Code
EOF
)

…hored

On wide monitors, centering the nav row inside the same max-w container as
page content left a large, growing gap before the logo. The header now uses
fixed padding only, so it stays a constant distance from the viewport edge
regardless of screen width. Also widens the /about header to max-w-7xl to
match the rest of the site.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@maddy-abrahams maddy-abrahams requested a review from nojibe June 18, 2026 21:41
@railway-app

railway-app Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

This PR was not deployed automatically as @maddy-abrahams does not have access to the Railway project.

In order to get automatic PR deploys, please add @maddy-abrahams to your workspace on Railway.

maddy-abrahams and others added 4 commits June 18, 2026 17:55
Matches the SiteHeader change: the /about header content was centered in
a max-w-7xl container, pushing the logo rightward on wide screens. Use
fixed padding only so the logo sits at the same left-edge position as the
home header.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Group About and Our Methodology with the Create button on the right side
instead of leaving the nav floating between the logo and the button.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Remove the redundant group-hover underline on the "Weval a Collective
Intelligence Project" lockup (the pointer cursor already signals that it
is clickable) and use a consistent hover:opacity-80 lightening on both
the home header and /about.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Shorten "Our Methodology" to "Methodology" and hide the "a Collective
Intelligence Project" tagline below the sm breakpoint so the nav stops
overflowing on small screens. Temporary fix ahead of a proper mobile
hamburger menu.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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