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

Range slider doesn't fire change event #8159

Closed
duizendnegen opened this Issue Jun 20, 2015 · 3 comments

Comments

Projects
None yet
3 participants
@duizendnegen

duizendnegen commented Jun 20, 2015

The range slider doesn't fire a change event when the lower bound is approaching the upper bound.

Imagine a slider with {min=1, max=5}. Now, the transition {lower=4, upper=5} to {lower=5, upper=5} happens visually, but no change event is fired. I've worked around this by listening to $(".slider").on("slidedrag"), then manually triggering a change event, but it's quite clumsy.

See the case live in a jsbin: http://jsbin.com/fajudodevi/1/edit?html,js,console,output - just drag the lower bound to 5.

@arschmitz

This comment has been minimized.

Show comment
Hide comment
@arschmitz

arschmitz Jun 21, 2015

Member

@duizendnegen Hi thank you for reporting this. It is indeed a bug

Member

arschmitz commented Jun 21, 2015

@duizendnegen Hi thank you for reporting this. It is indeed a bug

@arschmitz arschmitz added this to the 1.5.0 milestone Jun 21, 2015

cgack added a commit to cgack/jquery-mobile that referenced this issue Jun 25, 2015

cgack added a commit that referenced this issue Jul 9, 2015

@cgack

This comment has been minimized.

Show comment
Hide comment
@cgack

cgack Jul 9, 2015

Contributor

fixed with bcca12a landing on 1.5-dev branch

Contributor

cgack commented Jul 9, 2015

fixed with bcca12a landing on 1.5-dev branch

@cgack cgack closed this Jul 9, 2015

@duizendnegen

This comment has been minimized.

Show comment
Hide comment
@duizendnegen

duizendnegen commented Jul 9, 2015

👍

arschmitz added a commit to arschmitz/jquery-mobile that referenced this issue Jul 4, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment