fixup! feat(material/select): switch implementation to use MDC

Author: crisbeto

src/material/core/density/private/_all-density.scss

@@ -42,12 +42,12 @@
   // not possible as it would introduce a circular dependency for density because the `mat-core`
   // mixin that is transitively loaded by the `all-theme` file, imports `all-density` which
   // would then load `all-theme` again. This ultimately results a circular dependency.
+  @include private-all-unmigrated-component-densities($config);
   @include form-field-theme.density($config);
   @include input-theme.density($config);
   @include option-theme.density($config);
   @include optgroup-theme.density($config);
   @include select-theme.density($config);
-  @include private-all-unmigrated-component-densities($config);
 }
 
 

src/material/core/theming/_all-theme.scss

@@ -53,6 +53,7 @@
   @include icon-theme.theme($theme-or-color-config);
   @include list-theme.theme($theme-or-color-config);
   @include menu-theme.theme($theme-or-color-config);
+  // TODO: when migrating paginator, ensure that it is included *before* select.
   @include paginator-theme.theme($theme-or-color-config);
   @include progress-spinner-theme.theme($theme-or-color-config);
   @include radio-theme.theme($theme-or-color-config);
@@ -72,13 +73,13 @@
   $dedupe-key: 'angular-material-theme';
   @include theming.private-check-duplicate-theme-styles($theme-or-color-config, $dedupe-key) {
     @include core-theme.theme($theme-or-color-config);
+    @include private-all-unmigrated-component-themes($theme-or-color-config);
     @include card-theme.theme($theme-or-color-config);
     @include progress-bar-theme.theme($theme-or-color-config);
     @include tooltip-theme.theme($theme-or-color-config);
     @include form-field-theme.theme($theme-or-color-config);
     @include input-theme.theme($theme-or-color-config);
     @include select-theme.theme($theme-or-color-config);
-    @include private-all-unmigrated-component-themes($theme-or-color-config);
   }
 }
 

src/material/core/typography/_all-typography.scss

@@ -39,7 +39,6 @@
 
 @mixin private-all-unmigrated-component-typographies($config) {
   @include badge-theme.typography($config);
-  @include typography.typography-hierarchy($config);
   @include autocomplete-theme.typography($config);
   @include bottom-sheet-theme.typography($config);
   @include button-theme.typography($config);
@@ -54,6 +53,7 @@
   @include grid-list-theme.typography($config);
   @include icon-theme.typography($config);
   @include menu-theme.typography($config);
+  // TODO: when migrating paginator, ensure that it is included *before* select.
   @include paginator-theme.typography($config);
   @include progress-spinner-theme.typography($config);
   @include radio-theme.typography($config);
@@ -85,6 +85,7 @@
   // not possible as it would introduce a circular dependency for typography because the `mat-core`
   // mixin that is transitively loaded by the `all-theme` file, imports `all-typography` which
   // would then load `all-theme` again. This ultimately results a circular dependency.
+  @include typography.typography-hierarchy($config);
   @include private-all-unmigrated-component-typographies($config);
   @include card-theme.typography($config);
   @include progress-bar-theme.typography($config);

src/material/legacy-core/theming/_all-theme.scss

@@ -13,13 +13,13 @@
   $dedupe-key: 'angular-material-legacy-theme';
   @include theming.private-check-duplicate-theme-styles($theme-or-color-config, $dedupe-key) {
     @include core-theme.theme($theme-or-color-config);
+    @include all-theme.private-all-unmigrated-component-themes($theme-or-color-config);
     @include card-theme.theme($theme-or-color-config);
     @include progress-bar-theme.theme($theme-or-color-config);
     @include tooltip-theme.theme($theme-or-color-config);
     @include input-theme.theme($theme-or-color-config);
     @include form-field-theme.theme($theme-or-color-config);
     @include select-theme.theme($theme-or-color-config);
-    @include all-theme.private-all-unmigrated-component-themes($theme-or-color-config);
   }
 }
 

src/material/legacy-core/typography/_all-typography.scss

@@ -26,6 +26,7 @@
   // mixin that is transitively loaded by the `all-theme` file, imports `all-typography` which
   // would then load `all-theme` again. This ultimately results a circular dependency.
 
+  @include typography.typography-hierarchy($config);
   @include all-typography.private-all-unmigrated-component-typographies($config);
   @include option-theme.typography($config);
   @include optgroup-theme.typography($config);