Skip to content

Commit

Permalink
fix(button): use currentColor as icon fill for all states (#5270)
Browse files Browse the repository at this point in the history
  • Loading branch information
janhassel authored Feb 6, 2020
1 parent 1ac4f59 commit 0f9aef2
Showing 1 changed file with 1 addition and 22 deletions.
23 changes: 1 addition & 22 deletions packages/components/src/components/button/_button.scss
Original file line number Diff line number Diff line change
Expand Up @@ -36,11 +36,6 @@

.#{$prefix}--btn {
@include button-base;

&.#{$prefix}--btn--disabled > svg.#{$prefix}--btn__icon,
&:disabled > svg.#{$prefix}--btn__icon {
fill: $disabled-03;
}
}

// Reset intrisic padding in Firefox (see #731)
Expand Down Expand Up @@ -113,14 +108,6 @@
background: transparent;
color: $disabled;
outline: none;

> .#{$prefix}--btn__icon path {
fill: $disabled;
}
}

&:hover > .#{$prefix}--btn__icon path {
fill: $inverse-01;
}
}

Expand Down Expand Up @@ -156,10 +143,6 @@
&:hover,
&:active {
color: $hover-primary-text;

.#{$prefix}--btn__icon path {
fill: $hover-primary-text;
}
}

&:active {
Expand All @@ -176,10 +159,6 @@
background: transparent;
border-color: transparent;
outline: none;

.#{$prefix}--btn__icon path {
fill: $disabled;
}
}

&.#{$prefix}--btn--sm {
Expand Down Expand Up @@ -273,7 +252,7 @@
$danger,
$text-04,
$hover-danger,
$icon-03,
currentColor,
$active-danger
);

Expand Down

0 comments on commit 0f9aef2

Please sign in to comment.