-
-
Notifications
You must be signed in to change notification settings - Fork 267
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make the fixed navbar optional #536
base: master
Are you sure you want to change the base?
Conversation
Adds another on/off preference (default: on, keeps same behaviour) for the fixed navbar. When off the navbar will not remain at the top of the page when scrolling. This is useful for small displays such as phones where otherwise the navbar takes up a sizeable portion of the viewport.
ping @spikecodes @sigaloid Thanks for your contribution. Sorry it's taken us a while before we got a change to review. I'm unopposed to this and willing to bring this in (pending the merge conflict resolution), but I'll also let the other two project heads chime in to see what they say. |
I too am unopposed. 😄 🦀 thanks for your first contribution, I hope to see this go in soon! |
Introducing #662 into master has given rise to a merge conflict in this PR. I'll take care of it as part of the merge. |
I haven't forgotten about this. I'll try to get to this this week. |
58cf16e
to
48fd4be
Compare
Adds another on/off preference (default: on, keeps same behaviour) for the fixed navbar.
When off the navbar will not remain at the top of the page when scrolling.
This is useful for small displays such as phones where otherwise the navbar takes up a sizeable portion of the viewport.