refactor(material/core): remove unnecessary styles from mat-option (#21432)

crisbeto · mmalerba · commit 3eb016cbd770 · 2021-01-14T16:05:26.000-08:00
We had an override to make the option ripple semi-transparent in high contrast mode. This hasn't been necessary for a while, because we've been setting `display: none` on all ripples for high contrast users. (cherry picked from commit f12f6c3)
diff --git a/src/material-experimental/mdc-core/option/option.scss b/src/material-experimental/mdc-core/option/option.scss
@@ -61,11 +61,6 @@
     // user content and we don't want to disable mouse events on the user content.
     // Pointer events can be safely disabled because the ripple trigger element is the host element.
     pointer-events: none;
-
-    // Prevents the ripple from completely covering the option in high contrast mode.
-    @include cdk-high-contrast(active, off) {
-      opacity: 0.5;
-    }
   }
 }
 
diff --git a/src/material/core/option/option.scss b/src/material/core/option/option.scss
@@ -74,11 +74,6 @@
   // user content and we don't want to disable mouse events on the user content.
   // Pointer events can be safely disabled because the ripple trigger element is the host element.
   pointer-events: none;
-
-  // Prevents the ripple from completely covering the option in high contrast mode.
-  @include cdk-high-contrast(active, off) {
-    opacity: 0.5;
-  }
 }
 
 .mat-option-pseudo-checkbox {

Original file line number	Diff line number	Diff line change
`@@ -61,11 +61,6 @@`
`61`	`61`	`// user content and we don't want to disable mouse events on the user content.`
`62`	`62`	`// Pointer events can be safely disabled because the ripple trigger element is the host element.`
`63`	`63`	`pointer-events: none;`
`64`		`-`
`65`		`- // Prevents the ripple from completely covering the option in high contrast mode.`
`66`		`- @include cdk-high-contrast(active, off) {`
`67`		`- opacity: 0.5;`
`68`		`- }`
`69`	`64`	`}`
`70`	`65`	`}`
`71`	`66`