fix(button): Update colors to match guidance (#2598)

material-components · Apr 19, 2018 · 1be9d96 · 1be9d96
1 parent 099738c
commit 1be9d96
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/packages/mdc-button/_mixins.scss b/packages/mdc-button/_mixins.scss
@@ -128,8 +128,8 @@
 
   &:disabled {
     @include mdc-theme-prop(background-color, transparent);
-    @include mdc-theme-prop(color, text-disabled-on-light);
 
+    color: $mdc-button-disabled-ink-color;
     cursor: default;
     pointer-events: none;
   }
@@ -157,16 +157,16 @@
   border-style: solid;
 
   &:disabled {
-    @include mdc-theme-prop(border-color, text-disabled-on-light);
+    border-color: $mdc-button-disabled-ink-color;
   }
 }
 
 @mixin mdc-button--filled_() {
   @include mdc-button-horizontal-padding($mdc-button-contained-horizontal-padding);
 
   &:disabled {
-    @include mdc-theme-prop(background-color, rgba(black, .12));
-    @include mdc-theme-prop(color, text-disabled-on-light);
+    background-color: rgba(mdc-theme-prop-value(on-surface), .12);
+    color: $mdc-button-disabled-ink-color;
   }
 }
 

diff --git a/packages/mdc-button/_variables.scss b/packages/mdc-button/_variables.scss
@@ -18,3 +18,5 @@ $mdc-button-height: 36px;
 $mdc-button-horizontal-padding: 8px;
 $mdc-button-contained-horizontal-padding: 16px;
 $mdc-dense-button-height: 32px;
+
+$mdc-button-disabled-ink-color: rgba(mdc-theme-prop-value(on-surface), .37);