-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
@mixin mat-typography-level-to-styles #8700
Comments
I think it still makes sense to use the shorthand so we can save some bytes, but we can conditionally fall back to the individual properties if the font family is |
PR is #8721 |
Fixes the `mat-typography-level-to-styles` mixin generating an invalid style declaration if any of its value are set to `inherit`. Fixes angular#8700.
Fixes the `mat-typography-level-to-styles` mixin generating an invalid style declaration if any of its value are set to `inherit`. Fixes angular#8700.
Fixes the `mat-typography-level-to-styles` mixin generating an invalid style declaration if any of its value are set to `inherit`. Fixes #8700.
…lar#8721) Fixes the `mat-typography-level-to-styles` mixin generating an invalid style declaration if any of its value are set to `inherit`. Fixes angular#8700.
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. |
Bug, feature request, or proposal:
Bug or proposal
What is the current behavior?
https://github.com/angular/material2/blob/3b41c0c23e10fc74d9f687427f80c46f5da419c9/src/lib/core/typography/_typography-utils.scss#L32
What is the expected behavior?
What is the use-case or motivation for changing an existing behavior?
The shorthand
font
can't useinherit
forfont-family
The text was updated successfully, but these errors were encountered: