Skip to content

Keep the header at full height, and make its white background span the page - #241

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

mlaetitia merged 1 commit into
developfrom
fix/header-keep-height-develop

Conversation

@mlaetitia

Copy link
Copy Markdown
Collaborator

Related to #

Proposed changes

  • Keep the header at full height on scroll. Only the logo shrinks, so nothing below the header moves.
  • Make the white background span the full page width whenever the header is white, scrolled or with a megamenu open.
  • Centre the logo and nav in the scrolled header, which an unbalanced top padding was pushing low.

Why are these changes being made?

  • The header is sticky but still sits in flow, so shrinking it lifted the page and the nav appeared to jump on a small scroll.
  • The design asks for a full-width white background. The theme insets the page wrapper by 5vw above 1200px, and the background was painted on the header box, so it stopped short of both edges. It now sits on a layer that escapes that inset, the same way the megamenu panel already did.

Testing instructions

  • Above 1200px, scroll down: the white background reaches both edges, the header keeps its height, and only the logo shrinks.
  • Hover a megamenu item, scrolled and unscrolled: white veil and page dim as before, with its bottom border.
  • Check an AI template page scrolled: the green trim spans the page with the white.
  • Check a dropdown while scrolled: it opens instead of being clipped.

🤖 Generated with Claude Code

https://claude.ai/code/session_011rCHPCdbV5C5dDJjEmMDuy

…e page

The header is sticky but sits in flow, so shrinking it moved everything below it
and left the nav drifting up on a small scroll. It now holds its height and only
the logo shrinks, which also removes the content shift that made the class
toggle in a loop, so the scroll handler needs a single threshold again.

The white background moved off the header box onto a veil that escapes the
wrapper's 5vw inset, so it reaches both edges of the page as the design asks.
The megamenu panel already did this; both now share one mixin, and the AI
template's green trim rides along with it.

Along the way: the row is centred again now that nothing overrides the
unbalanced top padding, dropdowns are no longer clipped while scrolled, and the
1200px breakpoint no longer falls in both branches at once.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011rCHPCdbV5C5dDJjEmMDuy
@mlaetitia
mlaetitia merged commit 5226bf0 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