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
MdMenu : style mat-menu-panel #5870
Comments
You can set a class on the
|
thanks @crisbeto, it should be great to have a mixin for desktop use, where all elevations are minus 2. 8 is mobile menu by the way ;) |
FWIW, you can generate your own elevations via the |
Makes it a little easier to override the elevation of the `.mat-menu-panel`. Previously it wasn't possible to apply the `mat-elevation-z*` classes directly, because the elevation styles are defined earlier in the stylesheet than the menu styles. Fixes angular#5870.
Makes it a little easier to override the elevation of the `.mat-menu-panel`. Previously it wasn't possible to apply the `mat-elevation-z*` classes directly, because the elevation styles are defined earlier in the stylesheet than the menu styles. Fixes #5870.
@crisbeto While #5873 does make it easier to override, I think part of the original issue remains. The spec says menu elevations should be 6dp on desktop and 8dp on mobile, but the default is currently 2dp. EDIT: nvm, there's an inconsistency between the menu guidelines (2dp) and the elevation guidelines (6dp) |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
According to https://material.io/guidelines/material-design/elevation-shadows.html, Desktop resting elevation is 2dp below the listed values to accommodate mouse and non-touch environments.
For sample in the MdMenu component, we are unable to apply a mat-elevation-z6 class to the mat-menu-panel
The text was updated successfully, but these errors were encountered: