-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
fix(slider): change prop to stepMultiplier and deprecate stepMuliplier #5251
fix(slider): change prop to stepMultiplier and deprecate stepMuliplier #5251
Conversation
Deploy preview for carbon-elements ready! Built with commit 4aa9cbd |
Deploy preview for carbon-components-react failed. Built with commit 4aa9cbd https://app.netlify.com/sites/carbon-components-react/deploys/5e39a078607a20000800aefe |
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.
LGTM 👍 ✅
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.
LGTM 👍 - Thanks @abbeyhrt!
Closes #4897
This adds the deprecation warning to
stepMuliplier
and replaces it withstepMultiplier
and adds logic so that both the new and old prop implement the same behaviorChangelog
New
-
stepMultiplier
Changed
stepMuliplier
Testing / Reviewing
Check that
stepMultiplier
works the same way asstepMuliplier
.