diff options
author | Richard Worth <rdworth@gmail.com> | 2009-03-09 12:36:50 +0000 |
---|---|---|
committer | Richard Worth <rdworth@gmail.com> | 2009-03-09 12:36:50 +0000 |
commit | 439d93bf8b8092e57f4a6c22c049cd29427e0aad (patch) | |
tree | 5a16182926fdb6d4398be2d5bc4bb3c6098de506 /ui/ui.slider.js | |
parent | 3943e3d97cc501663f409dbacdef5f52c3fa0c3c (diff) | |
download | jquery-ui-439d93bf8b8092e57f4a6c22c049cd29427e0aad.tar.gz jquery-ui-439d93bf8b8092e57f4a6c22c049cd29427e0aad.zip |
Fixed #4292 (Slider: ui.value incorrect in slide event if handle at value 0)
Diffstat (limited to 'ui/ui.slider.js')
-rw-r--r-- | ui/ui.slider.js | 50 |
1 files changed, 33 insertions, 17 deletions
diff --git a/ui/ui.slider.js b/ui/ui.slider.js index 8f658abed..32b746689 100644 --- a/ui/ui.slider.js +++ b/ui/ui.slider.js @@ -299,7 +299,11 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, { }, _start: function(event, index) { - this._trigger("start", event, this._uiHash(index)); + this._trigger("start", event, { + handle: this.handles[index], + value: this.value(), + values: this.values() + }); }, _slide: function(event, index, newVal) { @@ -317,7 +321,11 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, { var newValues = this.values(); newValues[index] = newVal; // A slide can be canceled by returning false from the slide callback - var allowed = this._trigger("slide", event, this._uiHash(index, newVal, newValues)); + var allowed = this._trigger("slide", event, { + handle: this.handles[index], + value: newVal, + values: newValues + }); var otherVal = this.values(index ? 0 : 1); if (allowed !== false) { this.values(index, newVal, ( event.type == 'mousedown' && this.options.animate ), true); @@ -328,7 +336,10 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, { if (newVal != this.value()) { // A slide can be canceled by returning false from the slide callback - var allowed = this._trigger("slide", event, this._uiHash(index, newVal)); + var allowed = this._trigger("slide", event, { + handle: this.handles[index], + value: newVal + }); if (allowed !== false) { this._setData('value', newVal, ( event.type == 'mousedown' && this.options.animate )); } @@ -340,11 +351,27 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, { }, _stop: function(event, index) { - this._trigger("stop", event, this._uiHash(index)); + var uiHash = { + handle: this.handles[index], + value: this.value() + }; + if (this.options.values && this.options.values.length) { + uiHash.value = this.values(index) + uiHash.values = this.values() + } + this._trigger("stop", event, uiHash); }, _change: function(event, index) { - this._trigger("change", event, this._uiHash(index)); + var uiHash = { + handle: this.handles[index], + value: this.value() + }; + if (this.options.values && this.options.values.length) { + uiHash.value = this.values(index) + uiHash.values = this.values() + } + this._trigger("change", event, uiHash); }, value: function(newValue) { @@ -475,19 +502,8 @@ $.widget("ui.slider", $.extend({}, $.ui.mouse, { (oRange == "max") && (this.orientation == "vertical") && this.range[animate ? 'animate' : 'css']({ height: (100 - valPercent) + '%' }, { queue: false, duration: o.animate }); } - }, - - _uiHash: function(index, value, values) { - - var multiple = this.options.values && this.options.values.length; - return { - handle: this.handles[index], - value: value || (multiple ? this.values(index) : this.value()), - values: values || (multiple && this.values()) - }; - } - + })); $.extend($.ui.slider, { |