From b0d5f268fe22a32028c1c47e0cf9c7ff277e050e Mon Sep 17 00:00:00 2001 From: David Petersen Date: Tue, 16 Oct 2012 14:07:10 -0400 Subject: [PATCH] Slider: count the handles after setting up a range. Fixes #7922: Slider with range doesn't work if values are not set --- ui/jquery.ui.slider.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/ui/jquery.ui.slider.js b/ui/jquery.ui.slider.js index f24982878d8..9acf54a57ff 100644 --- a/ui/jquery.ui.slider.js +++ b/ui/jquery.ui.slider.js @@ -40,7 +40,7 @@ $.widget( "ui.slider", $.ui.mouse, { o = this.options, existingHandles = this.element.find( ".ui-slider-handle" ).addClass( "ui-state-default ui-corner-all" ), handle = "", - handleCount = ( o.values && o.values.length ) || 1, + handleCount, handles = []; this._keySliding = false; @@ -79,6 +79,8 @@ $.widget( "ui.slider", $.ui.mouse, { ( ( o.range === "min" || o.range === "max" ) ? " ui-slider-range-" + o.range : "" ) ); } + handleCount = ( o.values && o.values.length ) || 1; + for ( i = existingHandles.length; i < handleCount; i++ ) { handles.push( handle ); }