-
Notifications
You must be signed in to change notification settings - Fork 4
Docs/nav menu tooltip #54
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
✅ Deploy Preview for industrial-experience ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
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.
Info on tooltip behavior is fine, should remove all infos related to deprecated basic/map navigation.
Co-authored-by: Lukas Maurer <[email protected]>
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.
Looks good to me!
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.
A couple of questions here and a couple of changes suggested (mostly missing commas).
Co-authored-by: tokyojen <[email protected]>
Tooltip is now always visible (expand and collapse state), and additionally can be overridden with custom text e.g. when the label is truncated in expand state or needs additional context.
Additionally reworked the overall file structure a bit to be more consistent with other chapters