-
Notifications
You must be signed in to change notification settings - Fork 13
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
Update Navigation Design #430
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Thanks for the fix. the hover states are MUCH nicer now. The last little nit i have is that there shouldn't be text selection on the text for the main nav items. LGTM
@avatus Sorry for the re-request. Fixed a few things to get all CI/CD tests and checks to pass. All good now! |
CHANGES
Update Navigation design along the next and blog repos
Added box sizing and min width values to the docs layout body class to fix an issue with content overflowing on some viewports.
Skipped navigation data tests for now as those need to be updated to match the new setup, but we didn't have time to do that within this project's timeline
Preview
https://docs-git-tuukkatahtinen-navigationredesign-goteleport.vercel.app/docs/
Related Next repo PR: https://github.com/gravitational/next/pull/2344
Related Blog repo PR: https://github.com/gravitational/blog/pull/490