From 448b8a2c18843e5375f5b4000a8d5a9d37b6fdd0 Mon Sep 17 00:00:00 2001 From: Kristiyan Kostadinov Date: Tue, 13 Apr 2021 20:41:20 +0200 Subject: [PATCH] refactor(material/core): clean up prefixes from internal Sass symbols The Sass migration script doesn't migrate private symbols (starting with `_`) so we were left with some prefixes. These changes clean up the prefixes, as well as some unnecessary `!global` declarations. --- src/cdk/a11y/_index.scss | 6 +-- .../scrolling/virtual-scroll-viewport.scss | 6 +-- src/cdk/text-field/_index.scss | 9 ++-- .../mdc-button/_button-theme.scss | 54 +++++++++---------- .../mdc-form-field/_form-field-density.scss | 10 ++-- .../mdc-form-field/_form-field-theme.scss | 10 ++-- .../mdc-helpers/_focus-indicators-theme.scss | 6 +-- .../mdc-progress-bar/_progress-bar-theme.scss | 8 +-- .../_progress-spinner-theme.scss | 8 +-- .../mdc-tabs/_tabs-theme.scss | 16 +++--- .../autocomplete/_autocomplete-theme.scss | 4 +- src/material/badge/_badge-theme.scss | 12 ++--- .../bottom-sheet/_bottom-sheet-theme.scss | 4 +- .../bottom-sheet/bottom-sheet-container.scss | 14 ++--- src/material/button/_button-theme.scss | 32 +++++------ src/material/card/_card-theme.scss | 4 +- src/material/checkbox/_checkbox-theme.scss | 4 +- src/material/checkbox/checkbox.scss | 42 +++++++-------- src/material/chips/_chips-theme.scss | 22 ++++---- src/material/core/_core.scss | 8 +-- .../core/density/private/_compatibility.scss | 20 +++---- .../_focus-indicators-theme.scss | 6 +-- src/material/core/option/_optgroup-theme.scss | 4 +- src/material/core/option/_option-theme.scss | 4 +- .../_pseudo-checkbox-theme.scss | 4 +- .../pseudo-checkbox/pseudo-checkbox.scss | 12 ++--- src/material/core/theming/_theming.scss | 36 ++++++------- .../datepicker/_datepicker-theme.scss | 22 ++++---- src/material/datepicker/calendar-body.scss | 6 +-- src/material/datepicker/date-range-input.scss | 6 +-- src/material/dialog/_dialog-theme.scss | 4 +- src/material/divider/_divider-theme.scss | 4 +- .../form-field/_form-field-fill-theme.scss | 6 +-- .../form-field/_form-field-legacy-theme.scss | 16 +++--- .../form-field/_form-field-outline-theme.scss | 6 +-- .../form-field/_form-field-theme.scss | 6 +-- src/material/grid-list/_grid-list-theme.scss | 4 +- src/material/icon/_icon-theme.scss | 4 +- src/material/input/_input-theme.scss | 4 +- src/material/list/_list-theme.scss | 4 +- src/material/menu/_menu-theme.scss | 4 +- .../progress-bar/_progress-bar-theme.scss | 4 +- .../_progress-spinner-theme.scss | 4 +- .../progress-spinner/progress-spinner.scss | 9 ++-- src/material/radio/_radio-theme.scss | 12 ++--- src/material/select/_select-theme.scss | 4 +- src/material/sidenav/_sidenav-theme.scss | 4 +- .../slide-toggle/_slide-toggle-theme.scss | 12 ++--- src/material/slider/_slider-theme.scss | 12 ++--- src/material/snack-bar/_snack-bar-theme.scss | 4 +- src/material/sort/_sort-theme.scss | 4 +- src/material/table/_table-theme.scss | 4 +- src/material/tabs/_tabs-theme.scss | 20 +++---- src/material/toolbar/_toolbar-theme.scss | 18 +++---- src/material/tooltip/_tooltip-theme.scss | 4 +- 55 files changed, 284 insertions(+), 292 deletions(-) diff --git a/src/cdk/a11y/_index.scss b/src/cdk/a11y/_index.scss index 292804e3ec49..b1259c85fd0a 100644 --- a/src/cdk/a11y/_index.scss +++ b/src/cdk/a11y/_index.scss @@ -26,7 +26,7 @@ /// Emits the mixin's content nested under `$selector-context` if `$selector-context` /// is non-empty. /// @param selector-context The selector under which to nest the mixin's content. -@mixin _cdk-optionally-nest-content($selector-context) { +@mixin _optionally-nest-content($selector-context) { @if ($selector-context == '') { @content; } @@ -68,7 +68,7 @@ // Note that if this selector is updated, the same change has to be made inside // `_overlay.scss` which can't depend on this mixin due to some infrastructure limitations. .cdk-high-contrast-#{$target} { - @include _cdk-optionally-nest-content($selector-context) { + @include _optionally-nest-content($selector-context) { @content; } } @@ -76,7 +76,7 @@ @if ($encapsulation != 'off') { .cdk-high-contrast-#{$target} :host { - @include _cdk-optionally-nest-content($selector-context) { + @include _optionally-nest-content($selector-context) { @content; } } diff --git a/src/cdk/scrolling/virtual-scroll-viewport.scss b/src/cdk/scrolling/virtual-scroll-viewport.scss index 06c662a9e62d..c24aa19ed3ef 100644 --- a/src/cdk/scrolling/virtual-scroll-viewport.scss +++ b/src/cdk/scrolling/virtual-scroll-viewport.scss @@ -2,7 +2,7 @@ // their container element (e.g. ``, `