Skip to content

Commit

Permalink
fix(color-picker-slider-focus): add focus state to color picker slide…
Browse files Browse the repository at this point in the history
…r thumb

[Finishes #W-4172014]
  • Loading branch information
Jade Pennig committed Aug 2, 2017
1 parent dd85217 commit c29d368
Showing 1 changed file with 21 additions and 4 deletions.
25 changes: 21 additions & 4 deletions ui/components/color-picker/base/_index.scss
Original file line number Diff line number Diff line change
Expand Up @@ -231,10 +231,6 @@
$color-stops
);

&:focus {
outline: none;
}

@mixin thumb-styles {
appearance: none;
cursor: pointer;
Expand All @@ -245,6 +241,11 @@
background: $color-picker-slider-thumb-color-background;
}

@mixin thumb-styles-focus {
border-color: $color-border-input-active;
box-shadow: $shadow-button-focus;
}

&::-webkit-slider-thumb {
@include thumb-styles;
}
Expand All @@ -258,6 +259,22 @@
height: $color-picker-slider-height;
}

&:focus {
outline: none;

&::-webkit-slider-thumb {
@include thumb-styles-focus;
}

&::-moz-range-thumb {
@include thumb-styles-focus;
}

&::-ms-thumb {
@include thumb-styles-focus;
}
}

&::-moz-range-track {
height: 0;
}
Expand Down

0 comments on commit c29d368

Please sign in to comment.