-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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
[Slider] Make the slider work as intended when max%step !== 0 #18438
Conversation
Details of bundle changes.Comparing: 15e3336...c54accd
|
I am sorry for a bit of a mess this PR is, i'm first-time-contributor. |
Not a problem. If you don't know about |
Yeah, as I did not introduce any changes to the docs directly, i've decided not to touch the docs update hook. But then I figured out propTypes comments end up in docs too, thanks to your CI being awesome :) |
@macfire10 It's a great first pull request on Material-UI 👌🏻. Thank you for working on it! |
fixes #18424