Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Slider] is step broken ? #1842

Closed
BenNG opened this issue Oct 8, 2015 · 5 comments
Closed

[Slider] is step broken ? #1842

BenNG opened this issue Oct 8, 2015 · 5 comments
Labels
component: slider This is the name of the generic UI component, not the React module!

Comments

@BenNG
Copy link

BenNG commented Oct 8, 2015

Hi there !

I have this configuration

<Slider step={1} max={3} min={1} defaultValue={2}></Slider>

I was expected to have only 3 possible values 1,2 and 3.
But in real my slider can navigate through all the value.
Hope I did not miss something ...

Thanks for your work :))))))
@KapJI
Copy link
Contributor

KapJI commented Oct 9, 2015

I think my PR #1591 should fix that.

@KapJI
Copy link
Contributor

KapJI commented Oct 9, 2015

PR #1825 fixes that better.

@igorbt
Copy link
Contributor

igorbt commented Oct 15, 2015

@BenNG can you please check if bug was fixed by PR #1825 and eventually close the bug?

@BenNG
Copy link
Author

BenNG commented Oct 16, 2015

Hello,

I try to test but I have a lots of dependencies issues !
How can I test properly ?

@BenNG
Copy link
Author

BenNG commented Nov 3, 2015

Tested with material-ui:0.13.1 it works
I close :)
Thx !

@BenNG BenNG closed this as completed Nov 3, 2015
@oliviertassinari oliviertassinari added the component: slider This is the name of the generic UI component, not the React module! label Aug 22, 2020
mnajdova pushed a commit to mnajdova/material-ui that referenced this issue Nov 10, 2020
@dependabot-preview
Bump lint-staged from 10.0.8 to 10.2.7 (mui#1842)
23ff4f9 
Bumps [lint-staged](https://github.com/okonet/lint-staged) from 10.0.8 to 10.2.7.
- [Release notes](https://github.com/okonet/lint-staged/releases)
- [Commits](lint-staged/lint-staged@v10.0.8...v10.2.7)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component: slider This is the name of the generic UI component, not the React module!
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@BenNG @igorbt @oliviertassinari @KapJI