Conversation
The header shrank only after 100px of scrolling, and a single threshold meant the 25px the page lifts on shrinking could be handed back by scroll anchoring, re-crossing the bound and toggling the class repeatedly. The threshold to beat now depends on which state the header is in, so no correction can reach the other bound, and the handler runs once per animation frame on a passive listener. Mobile and desktop share one path; the mobile header no longer grows back mid-page on an upward scroll. max-height had no length to animate from, so the shrink snapped rather than eased; the base rules now set one, and the transitions name the properties that actually change. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011rCHPCdbV5C5dDJjEmMDuy (cherry picked from commit a3ef8d6)
…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 (cherry picked from commit 68bb1b1)
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 (cherry picked from commit 3c36769)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related to #
Proposed changes
Why are these changes being made?
Testing instructions
🤖 Generated with Claude Code
https://claude.ai/code/session_011rCHPCdbV5C5dDJjEmMDuy