Skip to main content

Bug Tracker

Side navigation

#1938 closed enhancement (fixed)

Opened November 16, 2007 06:43PM UTC

Closed February 08, 2008 06:46PM UTC

[PATCH] sliderMoveTo for multiple handles

Reported by: nostrademons Owned by: paul
Priority: major Milestone: 1.2.2
Component: ui 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 November 16, 2007 06:43PM UTC.

    Adds 'which' parameter to sliderMoveTo for multiple handles

Change History (2)

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.