feat(range-slider): add component (VIV-1488)

[RichardHelm]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (d61b119) to head (a11ca3c).
Report is 837 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##              main     #1584     +/-   ##
===========================================
  Coverage   100.00%   100.00%             
===========================================
  Files          123       316    +193     
  Lines         1562      5055   +3493     
  Branches       108       562    +454     
===========================================
+ Hits          1562      5055   +3493     

Flag	Coverage Δ
unittests	100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rinaok]

The navigation does not contain the component.

[rachelbt]

The navigation does not contain the component.

this is because we decided to have one or 2 teams checking the component before we add it to the "public" :)

[rachelbt]

looks good!
left 2 comments

[rachelbt]

The navigation does not contain the component.

this is because we decided to have one or 2 teams checking the component before we add it to the "public" :)

but you can see it with adding the component name:
https://vivid-fd8y50sq3b1b.deno.dev/components/range-slider

[TaylorJ76]

Looking good!
Only thing I noticed was that the screen reader (voiceover on mac) was not reading out the minus values correctly.
Eg: "-4" is being read as "4". I can see that the code is correct, but I wonder if we need to investigate why.

[RichardHelm]

@TaylorJ76 Well spotted, this seems to be an issue with VoiceOver: https://discussions.apple.com/thread/254773710?sortBy=best

It also affects other components like slider, number-field etc. so I think a possible fix / workaround is out of scope for this PR

Stale