-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Adding sub items to menu is confusing visually with all the + icons #18333
Comments
The plus is a trailing inserter when a sub-menu already exists. It should show only for the currently selected level. |
Work on #18310 should help alleviate some of this pain |
It feels as if this is a bug. I think I can remember that at some point all sub items behaved like the main items, and you could (like @mtias describes) only see the appender for the currently selected level. |
Yes, I found in git history that there was an issue with the registration of allowed blocks which prompted @talldan to hide menu items with css instead, but that leaves "on" all those appenders. I added this #19598 small PR to hide appenders for other navigation levels. Not sure if this is the best approach but if it is, it will help when we implement those flyout ideas in #18310. |
@draganescu Was this fixed by #19598? |
Yep, I think this is fixed by #19598. |
If you add a few sub-items you get this visual:
I would suggest that seeing just one is better and this isn't a great experience for people, beyond the background not letting me see the '+' any way that well.
The text was updated successfully, but these errors were encountered: