Skip to content

Commit fbe685c

Browse files
committed
fix(slider,slide-toggle): wrong cursor
Uses the `grab` cursor when hovering over an active slider's thumb and thumb labels, as well as on active slide toggles. Fixes #3778.
1 parent e964734 commit fbe685c

File tree

3 files changed

+23
-1
lines changed

3 files changed

+23
-1
lines changed

src/lib/core/style/_vendor-prefixes.scss

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,4 +28,9 @@
2828
cursor: -webkit-grab;
2929
cursor: grab;
3030
}
31+
32+
@mixin cursor-grabbing {
33+
cursor: -webkit-grabbing;
34+
cursor: grabbing;
35+
}
3136
/* stylelint-enable */

src/lib/slide-toggle/slide-toggle.scss

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,9 @@ $mat-slide-toggle-bar-track-width: $mat-slide-toggle-bar-width - $mat-slide-togg
105105

106106
// Once the thumb container is being dragged around, we remove the transition duration to
107107
// make the drag feeling fast and not delayed.
108-
&.mat-dragging {
108+
&.mat-dragging, &:active {
109+
@include cursor-grabbing;
110+
109111
transition-duration: 0ms;
110112
}
111113
}

src/lib/slider/slider.scss

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
@import '../core/style/variables';
2+
@import '../core/style/vendor-prefixes';
23

34

45
// This refers to the thickness of the slider. On a horizontal slider this is the height, on a
@@ -97,7 +98,19 @@ $mat-slider-focus-ring-size: 30px !default;
9798
}
9899
}
99100

101+
%_mat-slider-cursor {
102+
.mat-slider:not(.mat-slider-disabled) & {
103+
@include cursor-grab;
104+
}
105+
106+
.mat-slider:not(.mat-slider-disabled) &:active {
107+
@include cursor-grabbing;
108+
}
109+
}
110+
100111
.mat-slider-thumb {
112+
@extend %_mat-slider-cursor;
113+
101114
position: absolute;
102115
right: -$mat-slider-thumb-size / 2;
103116
bottom: -$mat-slider-thumb-size / 2;
@@ -113,6 +126,8 @@ $mat-slider-focus-ring-size: 30px !default;
113126
}
114127

115128
.mat-slider-thumb-label {
129+
@extend %_mat-slider-cursor;
130+
116131
display: none;
117132
align-items: center;
118133
justify-content: center;

0 commit comments

Comments
 (0)