-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Improve navigation styling and experience #856
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
Conversation
|
@jekyllbot: merge +major |
|
sorry, but wanted to ask about this: it looks horrible on mobile screens, what's the point of making that change? Edit: made the issue about that #860 |
I know you mean well, but FWIW, I hate this behavior on websites, and have to spend a lot of time adding an adblocker rule on each site I visit to kill these pointless sticky headers in order to free up screen space so I can read the actual content. I'm far from the only one either, as you can see from the discussion here for instance. Making this behavior the default on a widely used website theme seems like it's going to cause a lot of pain for people. |
|
Hello @Storyyeller, thank you for reaching out and voicing your disapproval. |
|
Hello everyone, |
|
@Storyyeller FYI: I have reverted the sticky-header to previous default state via dfaea3e. |
Summary
nav-items.htmlto render navigation link-items for all screen-sizes.header_pagesconfig data into theminimanamespace asminima:nav_pages