Skip to content

Stop the header moving when it scrolls - #243

Merged
mlaetitia merged 1 commit into
developfrom
fix/header-stop-movement-develop
Sep 25, 2026
Merged

mlaetitia merged 1 commit into
developfrom
fix/header-stop-movement-develop

Conversation

@mlaetitia

Copy link
Copy Markdown
Collaborator

Related to #

Proposed changes

  • Hold the logo column at a fixed width so the menu no longer reflows from two rows to one when the logo shrinks.
  • Offset the header by the admin bar height in both states, so logged-in users no longer see it drop 32px on scroll.
  • Drop the leftover link re-padding in the scrolled state.

Why are these changes being made?

  • The header is meant to hold still on scroll, with only the logo shrinking. Three things still moved: the logo column gave its width to the nav as the logo shrank, which pushed "Get Involved" onto the first row and shifted the whole menu; the admin bar offset applied only once the header went small; and the menu links were re-padded, left from when the header itself got shorter.

Testing instructions

  • Above 1200px, scroll down and back up: only the logo changes size. The menu keeps its rows and nothing shifts.
  • Do the same logged in, with the admin bar showing.

🤖 Generated with Claude Code

https://claude.ai/code/session_011rCHPCdbV5C5dDJjEmMDuy

Three things still shifted when the header went small. The logo column took its
width from the logo, so shrinking the logo handed the nav that width and the
menu reflowed from two rows to one. The admin bar offset applied only to the
small state, so logged-in users saw the whole header drop 32px the moment the
class flipped. And the menu links were re-padded, left over from when the header
itself got shorter.

The column now holds its width, the offset applies in both states, and the link
padding is gone. Only the logo changes, which is the point.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011rCHPCdbV5C5dDJjEmMDuy
@mlaetitia
mlaetitia merged commit 784b021 into develop Sep 25, 2026
5 checks passed
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