Repository navigation
Offset in-page anchor jumps below the fixed navbar - #2026
Merged
Merged
Conversation
Add scroll-padding-top to the html element so that linking to an in-page anchor scrolls the target below the sticky navbar instead of hiding it behind it. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Member
Author
|
@USRSE/usrse-maintainers Please review. This affects the entire site, but seems to work better, at least in my limited testing. Feel free to ignore or provide a better fix though. |
1 of 2 tasks
mrmundt
approved these changes
Sep 18, 2026
This was referenced Sep 18, 2026
Member
Author
|
Follow-up for anyone landing here later: this didn't fully resolve the anchor-scroll issue (#1483) in production — it turned out |
exoticDFT
pushed a commit
that referenced
this pull request
Sep 30, 2026
PR #2026 added assets/css/style.scss to set scroll-padding-top, but the repo already has a hand-maintained assets/css/style.css that compiles to the same output path. Jekyll's production build was serving the static style.css and silently dropping the Sass-compiled file, so the new rule never reached us-rse.org even though the PR merged and Pages rebuilt successfully. Move the rule directly into the static style.css and remove the now unnecessary style.scss. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
3 tasks done
exoticDFT
pushed a commit
that referenced
this pull request
Sep 30, 2026
style.css is a pre-compiled Bulma snapshot with hand edits and no Sass build, which isn't obvious to contributors. A sibling style.scss silently shadowed it in #2026. Add a header comment and a docs/details.md section explaining how to change styles, plus a CI job that fails when a Sass file would compile to the path of an existing static .css file. Part of #2063 (phases 1 and 2). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
2 participants
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.
Description
Adds a
scroll-padding-toprule on thehtmlelement so that following an in-page anchor link scrolls the target below the fixed (sticky) navbar instead of leaving it hidden behind the navbar. The offset (3.25rem) matches the Bulma fixed-navbar height, and applies to in-page anchor jumps site-wide.Checklist:
This is a small style/functionality fix with no content changes.
When you are ready for a technical review/merge, post the for the link for the PR in the US-RSE Slack (#website) to ask for reviewers.