Skip to content
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

fix(multiple): account for mixed declarations in latest Sass version #29596

Merged
merged 1 commit into from
Aug 20, 2024

Commits on Aug 20, 2024

  1. fix(multiple): account for mixed declarations in latest Sass version

    Fixes a ton of warnings that showed up in the latest version of Sass as a result of https://sass-lang.com/documentation/breaking-changes/mixed-decls/.
    
    BREAKING CHANGE:
    * In order for Material to be compatible with [recent changes in Sass](https://sass-lang.com/documentation/breaking-changes/mixed-decls/) and upcoming changes in the CSS standard, tokens are now emitted in-place, rather the being hoisted to the top of the selector. As a result, some token overrides might not apply anymore. This is relevant primarily for the cases like `@include mat.button-theme($theme); --mat-button-color: red;`. It can be resolved by wrapping the overrides with `& {}`, for example `@include mat.button-theme($theme); & { --mat-button-color: red; }`.
    
    Fixes angular#29591.
    crisbeto committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    e6a0f28 View commit details
    Browse the repository at this point in the history