Skip to content
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

Add sticky tab navigation #162

Merged
merged 2 commits into from Jul 26, 2019
Merged

Add sticky tab navigation #162

merged 2 commits into from Jul 26, 2019

Conversation

lgeiger
Copy link
Member

@lgeiger lgeiger commented Jul 25, 2019

I used my rusty HTML/CSS skills, to play a bit with our documentation site:
This changes the top level navigation have a persistent tab bar:
Bildschirmfoto 2019-07-26 um 00 07 24

One thing I am not sure about is if it is good to repeat the top-level navigation on the left sidebar or should we switch that back to how it is currently done in https://larq.dev/?

@lgeiger lgeiger added the documentation Issues related to missing or wrong documentation label Jul 25, 2019
@MariaHeuss
Copy link
Contributor

I have a week preference on having less information/text on one site, therefore I would go with the persistent tab bar but the side bar as it is currently done. But as I said no strong preference.

@koenhelwegen
Copy link
Contributor

Agree with Maria, the sticky top bar is great but would keep left panel as it is now.

@lgeiger
Copy link
Member Author

lgeiger commented Jul 26, 2019

OK, I'll take a look and see if I can figure it out how to make it work.

@lgeiger
Copy link
Member Author

lgeiger commented Jul 26, 2019

I simplified the side navigation to not include the top level:
Bildschirmfoto 2019-07-26 um 14 48 29

@lgeiger lgeiger merged commit 48d9fed into master Jul 26, 2019
@lgeiger lgeiger deleted the docs-sticky-header branch July 26, 2019 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Issues related to missing or wrong documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants