#11503 closed defect (fixed)
dojox.form.RangeSlider - Max-Slider Mouseclick
Reported by: | Ferdinand Weinberger | Owned by: | dante |
---|---|---|---|
Priority: | high | Milestone: | 1.5.1 |
Component: | DojoX Form | Version: | 1.5 |
Keywords: | dojox rangeslider | Cc: | |
Blocked By: | Blocking: |
Description
the max-slider of the rangesliders can only be moved by scroll and keyboard. if you click on it it will not move and the console will throw an error "widget is undefined.." best example is the testpage: http://archive.dojotoolkit.org/nightly/dojotoolkit/dojox/form/tests/test_RangeSlider.html
i found a workaround, but maybe the whole widget should be checked by its creator.
by adding the line dijit.form._SliderMoverMax.prototype.widget = _self; to line 30 of dojox/form/RangeSlider.js it seems to be solved
Line 27 - 34
// define a custom constructor for a SliderMoverMax that points back to me var _self = this; var mover = function(){ dijit.form._SliderMoverMax.prototype.widget = _self; dijit.form._SliderMoverMax.apply(this, arguments); this.widget = _self; }; dojo.extend(mover, dijit.form._SliderMoverMax.prototype);
Change History (4)
comment:1 Changed 11 years ago by
comment:3 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:4 Changed 10 years ago by
Milestone: | tbd → 1.5.1 |
---|
Note: See
TracTickets for help on using
tickets.
Dragging the whole silder also generates an exception. A similar fix needs to be made a few lines lower:
On line 41 add
dijit.form._SliderBarMover.prototype.widget = _self;
like this:
Also the code is using a non-existant function: dijit.form._FormValueWidget.prototype._onKeyPress.apply(this, arguments); in the _onKeyPress function. Whenever you type a key that isn't handled by the RangSilder? you get an error. I just commented out the call and there didn't seem to be any negative effects. If its still needed then an alternate function needs to be found.