Skip to content

Commit 8cb98c6

Browse files
authored
fix(slider): offset variant border radius bug fix (#3611)
* feat(slider): offset variant border radius bug fix * feat(slider): fix range slider
1 parent 3aec28a commit 8cb98c6

File tree

3 files changed

+18
-0
lines changed

3 files changed

+18
-0
lines changed

.changeset/rude-jokes-judge.md

+9
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
---
2+
"@spectrum-css/slider": minor
3+
---
4+
5+
# Slider: offset variant track fix
6+
7+
The border radius styles were not being applied to the second instance of the `spectrum-Slider-track` when the offset variant is activated. The reason for this bug is because when the `offset` is selected, the template structure changes as `spectrum-Slider-fill` gets added to the slider.
8+
9+
Adding a sibling combinator `&~.spectrum-Slider-track` to `spectrum-Slider-track` when offset is activated resolved the issue.

components/slider/dist/metadata.json

+1
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,7 @@
6262
".spectrum-Slider-ticks ~ .spectrum-Slider-handleContainer .spectrum-Slider-handle",
6363
".spectrum-Slider-track",
6464
".spectrum-Slider-track ~ .spectrum-Slider-track",
65+
".spectrum-Slider-track ~ .spectrum-Slider-track:before",
6566
".spectrum-Slider-track:before",
6667
".spectrum-Slider-track:first-of-type:before",
6768
".spectrum-Slider-track:last-of-type:before",

components/slider/index.css

+8
Original file line numberDiff line numberDiff line change
@@ -537,6 +537,12 @@
537537
&::before {
538538
background: var(--highcontrast-slider-track-color-static, var(--mod-slider-track-color, var(--spectrum-slider-track-color)));
539539
}
540+
541+
/* Styles below are applied to the sibling spectrum-Slider-track when filled-offset variant is activated */
542+
& ~ .spectrum-Slider-track::before {
543+
border-start-end-radius: var(--mod-slider-track-corner-radius, var(--spectrum-slider-track-corner-radius));
544+
border-end-end-radius: var(--mod-slider-track-corner-radius, var(--spectrum-slider-track-corner-radius));
545+
}
540546
}
541547

542548
/* All variants other than filled-offset get a new track color for highcontrast mode */
@@ -621,6 +627,8 @@
621627
.spectrum-Slider-track {
622628
&:not(:first-of-type, :last-of-type)::before {
623629
background: var(--highcontrast-slider-filled-track-fill-color, var(--mod-slider-track-fill-color, var(--spectrum-slider-track-fill-color)));
630+
border-start-end-radius: 0;
631+
border-end-end-radius: 0;
624632
}
625633
}
626634
}

0 commit comments

Comments
 (0)