Skip to content

Commit

Permalink
fix(material/core): remove core style parameters (#25738)
Browse files Browse the repository at this point in the history
* fix(material/core): remove core style parameters

* fix(material/core): dev-app theme
  • Loading branch information
andrewseguin committed Oct 3, 2022
1 parent 67e3e3f commit e73ba59
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 6 deletions.
2 changes: 1 addition & 1 deletion integration/yarn-pnp-compat/src/styles.scss
Expand Up @@ -8,7 +8,7 @@ $theme: mat.define-light-theme((
density: 0
));

@include mat.core($theme);
@include mat.core();

@include mat.all-legacy-component-themes($theme);
@include mat.all-component-themes($theme);
Expand Down
2 changes: 1 addition & 1 deletion src/dev-app/theme.scss
Expand Up @@ -21,7 +21,7 @@ $candy-app-theme: mat.define-light-theme((
// Include the common styles for Angular Material. We include this here so that you only
// have to load a single css file for Angular Material in your app.
// **Be sure that you only ever include this mixin once!**
@include mat.core($candy-app-theme);
@include mat.core();

// Include the default theme styles.
@include mat.all-component-themes($candy-app-theme);
Expand Down
3 changes: 1 addition & 2 deletions src/material/core/_core.scss
Expand Up @@ -5,8 +5,7 @@
@use './focus-indicators/private';

// Mixin that renders all of the core styles that are not theme-dependent.
// TODO: Remove the mixin's parameters after all internal clients migrated
@mixin core($typography-config: null, $exclude-typography: false) {
@mixin core() {
@include ripple.ripple();
@include cdk.a11y-visually-hidden();
@include cdk.overlay();
Expand Down
3 changes: 1 addition & 2 deletions src/material/legacy-core/_core.scss
Expand Up @@ -5,10 +5,9 @@
@use '../core/focus-indicators/private';

// Mixin that renders all of the core styles that are not theme-dependent.
// TODO: Remove the mixin's parameters after all internal clients migrated
/// @deprecated Use `mat.core` instead. See https://material.angular.io/guide/mdc-migration for information about migrating.
/// @breaking-change 17.0.0
@mixin core($typography-config: null, $exclude-typography: false) {
@mixin core() {
@include ripple.ripple();
@include cdk.a11y-visually-hidden();
@include cdk.overlay();
Expand Down

0 comments on commit e73ba59

Please sign in to comment.