Skip to main content

Search and Top Navigation

#1938 closed enhancement (fixed)

Opened November 16, 2007 06:43PM UTC

Closed February 08, 2008 06:46PM UTC

Last modified February 26, 2009 11:17AM UTC

[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 (0)
Change History (4)

Changed November 19, 2007 01:57PM UTC by paul comment:1

owner: → paul
status: newassigned

Changed February 08, 2008 06:46PM UTC by paul comment:2

resolution: → fixed
status: assignedclosed

implemented in 1.5b.

Changed May 24, 2008 03:39AM UTC by comment:3

milestone: 1.2.2

Milestone 1.2.2 deleted

Changed February 26, 2009 11:17AM UTC by paul comment:4

milestone: → 1.5