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 comment:1
owner: | → paul |
---|---|
status: | new → assigned |
Changed February 08, 2008 06:46PM UTC by comment:2
resolution: | → fixed |
---|---|
status: | assigned → closed |
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 comment:4
milestone: | → 1.5 |
---|