Add announcement.position (above / below navbar) - #286
Merged
Conversation
Appending the banner after .quarto-secondary-nav broke the secondary nav's mobile headroom positioning: when the fixed header unpins with translateY(-100%), the secondary nav counter-translates by its own height to stay pinned, which only lands at the viewport top when it is the header's last child. Inserting the banner before the secondary nav (instead of appending) keeps that invariant while still rendering the banner below the navbar.
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.
This PR adds an
announcement.positionconfig key that lets users choose whether the site-wide announcement banner renders above or below the navbar.above-navbar
below-navbar
closes #273.