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

Rangeslider layout breaks upon form reset #5773

Closed
gabrielschulhof opened this Issue Mar 15, 2013 · 1 comment

Comments

Projects
None yet
3 participants
@gabrielschulhof
Contributor

gabrielschulhof commented Mar 15, 2013

Steps to reproduce:

  1. Visit http://jsbin.com/onibuc/350/
  2. Type 50 into the lower end.
  3. Click the "Reset" button

At this point the range slider layout breaks.

@agcolom

This comment has been minimized.

Show comment
Hide comment
@agcolom

agcolom Mar 19, 2013

Member

ok, it seems that when the reset occurs, the css for "ui-slider-bg" is wrong. it gets a width of the start of the slider to the second handle, drawn from the first handle, instead of getting the width from the first handle to the second handle, drawn from the first handle. (so on reset, we need to get the value of the width by doing value2 - value1 instead of just value2)

Is it that _updateHighlight(); is not called upon reset?

Member

agcolom commented Mar 19, 2013

ok, it seems that when the reset occurs, the css for "ui-slider-bg" is wrong. it gets a width of the start of the slider to the second handle, drawn from the first handle, instead of getting the width from the first handle to the second handle, drawn from the first handle. (so on reset, we need to get the value of the width by doing value2 - value1 instead of just value2)

Is it that _updateHighlight(); is not called upon reset?

@ghost ghost assigned arschmitz Mar 21, 2013

@arschmitz arschmitz closed this in ef3a3f8 Apr 1, 2013

arschmitz added a commit that referenced this issue Apr 1, 2013

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