#9046 closed bug (fixed)

Opened February 01, 2013 11:13AM UTC

Closed March 18, 2015 02:49PM UTC

Last modified March 18, 2015 02:49PM UTC

Slider: Setting both values of range slider to maximum value, locks the slider

Reported by: sup Owned by:
Priority: minor Milestone: 1.12.0
Component: ui.slider Version: 1.10.0
Keywords: Cc:
Blocked by: Blocking:
Description

The slider handles are locked, after setting both values of range slider to the maximum value. Worked fine with the previous versions (tested 1.8.23 and 1.9.2)

Test case: http://jsbin.com/oyires/1

Attachments (0)
Change History (10)

Changed February 01, 2013 01:03PM UTC by petersendidit comment:1

status: new → open
summary: Setting both values of range slider to maximum value, locks the slider → Slider: Setting both values of range slider to maximum value, locks the slider

Changed February 18, 2013 05:06PM UTC by sup comment:2

_comment0: There is a workaround for this issue, set the values separately (from hight to low index) \ \ Extended test case: http://jsbin.com/oyires/4/ → 1361264320167390
_comment1: There is a workaround for this issue, set the values separately (from hight to low index) \ \ {{{ \ // original code \ handle.slider('option', { values: [from, to] }); \ \ // workaround \ handle.slider('values', 1, to); \ handle.slider('values', 0, from); \ }}} \ \ Extended test case: http://jsbin.com/oyires/4/ → 1363600249065850

There is a workaround for this issue, set the values separately (from hight to low index)

// original code
handle.slider('option', { values: [from, to] });

// workaround
handle.slider('values', 1, to);
handle.slider('values', 0, from);

Extended test case: http://jsbin.com/oyires/10/

Changed April 09, 2013 12:06AM UTC by lucas.ams comment:3

_comment0: I just fixed this bug. \ Basically, by the time both values are put to the maximum value, I gave a focus on the first handle (doing that we avoid the lock). \ I'll send a pull request. → 1365871623348468

I just fixed this bug.

Basically, by the time both handles are set to the maximum value, I give a focus on the first handle. Doing that, the lock is avoided because the user will have control on it (first handle), allowing drag it to the left whenever necessary.

I'll send a pull request.

Changed April 13, 2013 11:38PM UTC by lucas.ams comment:4

I've just commited: https://github.com/lams1986/jquery-ui/commit/3a6feb7caf7fd7d8d63c268380fabf5b17bab254

Changed June 05, 2014 02:08PM UTC by scottgonzalez comment:5

#10091 is a duplicate of this ticket.

Changed February 12, 2015 03:11PM UTC by sup comment:6

Looks good to me, please close the ticket.

Thank you!

Changed February 12, 2015 03:32PM UTC by scottgonzalez comment:7

Replying to [comment:6 sup]:

Looks good to me, please close the ticket.

Why? This hasn't been fixed.

Changed March 16, 2015 11:16AM UTC by jzaefferer comment:8

Pull request: https://github.com/jquery/jquery-ui/pull/1502

Changed March 18, 2015 02:49PM UTC by Ablay Keldibek comment:9

resolution: → fixed
status: open → closed

Slider: Fix handle order when setting values of range slider to max

When both values are set to the maximum change them in descending order

Fixes #9046

Closes gh-1502

Changeset: dff1c74dd4c0fd6b951c7c183bebae09f9f229f6

Changed March 18, 2015 02:49PM UTC by scottgonzalez comment:10

milestone: none → 1.12.0