-
Notifications
You must be signed in to change notification settings - Fork 201
fix: ramp slider (#829) #923
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Commit message needs to include:
BREAKING CHANGE: Slider color variant removed, used Color Slider component
Also need migration guide.
8069fd9
to
ba0622b
Compare
Good points @lazd .
I used instead |
VRT successfully! 🎊 |
The disabled color seems to have changed for ramp slider. Is that a result of the tokens changing? |
BREAKING CHANGE: Color slider is now a separate component
ba0622b
to
50fca35
Compare
VRT successfully! 🎊 |
VRT successfully! 🎊 |
Adding back ramp-slider and removing color-slider.
Description
Addresses issue #829
How and where has this been tested?
URL: http://localhost:3000/docs/slider.html
Chrome Version 85.0.4183.102 (Official Build) (64-bit) on MacOS
Screenshots
To-do list