-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
MudMenu: Fix submenu activators being too wide #11367
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
MudMenu: Fix submenu activators being too wide #11367
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #11367 +/- ##
==========================================
- Coverage 91.11% 91.10% -0.01%
==========================================
Files 465 465
Lines 14407 14407
Branches 2789 2788 -1
==========================================
- Hits 13127 13126 -1
+ Misses 643 642 -1
- Partials 637 639 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Review or wait? I noticed you changed it to draft. |
|
You're right, but these dense menus don't strictly follow MD as they are too small in general. I would like to change that in the next major |
|
|
I did not realize that is just Dense. I will say a Dense icon padding and extra width should likely be this way. |




Description
The submenu in the example was taking up too much space. Changing the display type is the lazy fix.
How Has This Been Tested?
visually
Type of Changes
Before:

After:

Checklist
dev).