-
Notifications
You must be signed in to change notification settings - Fork 63
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 Menu hover state #1410
Update Menu hover state #1410
Conversation
But does not work on complex menu
But not ideal, width still auto adjusts on complex menu
So this update ended up a bit more complex than expected. We have auto width menus so the width gets auto adjusted when text becomes bold on hover. I added a hack to avoid the auto width adjustment but this only works for simple menus (when there's only one child and the child is a string). Example of menu working as expected: Example of complex menu not working ideal (as there's more than one child and combination of different types): |
for complex menus
Testing with same as menu title color I think it looks better
Blocked by #1426 |
Also added radius for first and last menu list item
Changes done as suggested in #1426 and ready for review now |
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.
LGTM 👍
Resolves #1363