feat(material-experimental/mdc-menu): support density
Adds support for density to the MDC-based mat-menu.
@jelbourn Do you know if we want to change density of things in an overlay? I remember there being some discussion about not needing to do that, but also maybe it looks weird if the trigger is dense and the popup isnt?
it looks like the spec does show a dense option too: https://material.io/components/menus#specs
https://material.io/design/layout/applying-density.html#usage
Don’t increase the density of components that involve focused tasks, such as interacting with a dropdown menu or picker. Increasing density on components such as date pickers reduces usability by limiting tappable space.
That does seem to conflict with the menu spec. We should raise it to the Material folks
I added an agenda item for our next meeting with them. Will mark this as blocked pending their feedback
I filed a bug with the spec
Did we ever get an answer on this?
Closing this for now given how much the menu has changed since then and how out of date it is.
This issue has been automatically locked due to inactivity. Please file a new issue if you are encountering a similar or related problem.
Read more about our automatic conversation locking policy.
This action has been performed automatically by a bot.