Opened 16 years ago

Closed 15 years ago

Last modified 14 years ago

#1938 closed enhancement (fixed)

[PATCH] sliderMoveTo for multiple handles

Reported by: nostrademons Owned by: paul
Priority: major Milestone: 1.5
Component: ui.core Version: 1.2.1
Keywords: slider Cc:
Blocked by: Blocking:

Description

There're several folks asking for multiple-handle sliderMoveTo on the UI list:

http://groups.google.com/group/jquery-ui/browse_thread/thread/20db89aaad9836f0#

Kalath e-mailed me off-list and asked about it, and I wrote up a quick patch. It adds an extra parameter to sliderMoveTo which is the (zero-based) index of the handle to move, so to move the 3rd handle to 40 you'd do:

$('#slider').sliderMoveTo(40, null, null, null, 2);

The patch moves most of the body of moveTo() into an inner function that takes the interaction handle, and then calls that function with either this.interaction (single handle) or this.interactions[which] (multiple handles). All the rest of the touched lines are indentation changes or this->self swaps to make the moveTo body work inside an inner function.

It's dependent upon bug #1737. The patches themselves are independent (and both should be applied), but sliderMoveTo doesn't work at all until #1737 is fixed.

Attachments (1)

bug1938.patch (2.8 KB) - added by nostrademons 16 years ago.
Adds 'which' parameter to sliderMoveTo for multiple handles

Download all attachments as: .zip

Change History (5)

Changed 16 years ago by nostrademons

Attachment: bug1938.patch added

Adds 'which' parameter to sliderMoveTo for multiple handles

comment:1 Changed 16 years ago by paul

Owner: set to paul
Status: newassigned

comment:2 Changed 15 years ago by paul

Resolution: fixed
Status: assignedclosed

implemented in 1.5b.

comment:3 Changed 15 years ago by (none)

Milestone: 1.2.2

Milestone 1.2.2 deleted

comment:4 Changed 14 years ago by paul

Milestone: 1.5
Note: See TracTickets for help on using tickets.