-
Notifications
You must be signed in to change notification settings - Fork 5
Docs navigation update #297
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
base: main
Are you sure you want to change the base?
Conversation
|
Amplify deployment status
|
7d7b709 to
33786c2
Compare
…t, remove unnecessary swizzled components
.gitmodules
Outdated
| url = https://github.com/aatuvai/teleport/ | ||
| branch = aatuvai/docs-header |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Revert before merge to main
|
@aatuvai I just updated my PR to configure the docs for multiple sidebars: I think this looks really nice with the top nav, though it doesn't seem like there's a way to navigate between sidebars on mobile. Is that expected? Thanks! |
Thanks for catching this @ptgott! I'll make sure to check that navigating between sidebars works on mobile. |
f789637 to
384b7a4
Compare
|
@aatuvai Should we put this in draft until mobile navigation is ready? |
|
@ptgott I converted it to draft |
|
@ptgott I added the global sidebar to the config in my fork so it at least shows up on the home page. However I'm not sure how to solve the issue with not being able to switch the sidebars once you go to sub pages. Do you know if there's a way to use the global sidebar config for mobile only and use the individual sidebars on the desktop view? |
Per the Docusaurus docs, Docusaurus associates each page with a sidebar using the sidebar configuration, and you can manually specify a sidebar for a page using the We may need a component that allows users to switch sidebars on mobile. |
This PR adds the new docs navigation bar for the revamp.
Closes #279
Closes #170