#2617 closed bug (fixed)
ui.slider.js range focus bug.
Reported by: | norbert | Owned by: | eduardo |
---|---|---|---|
Priority: | major | Milestone: | 1.5 |
Component: | ui.core | Version: | 1.2.3 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
If you go to the third example jQuery UI slider demo page (http://dev.jquery.com/view/trunk/ui/demos/ui.slider.html), move the first handle a couple of notches to the right and then click to the left or right of that first handle (i.e. to move it again), a JavaScript error occurs.
Changing the following line of the ui.slider.js focus function apparently solves the problem:
From: if(hard) this.currentHandle.parent()[0].focus();
To: if(hard && this.currentHandle.parent()[0]) this.currentHandle.parent()[0].focus();
Clicking on the slider range or the slider background without clicking first on one of the handles triggers other JavaScript errors too.
I have the feeling that the slider functionality mostly been tested using single-handle slider scenarios (which is probably what the slider will be used for 90% of the time). As a result, some functionality breaks when two handles and a range are present.
Regards,
Norbert.
Change History (4)
comment:1 Changed 15 years ago by
Owner: | changed from paul to braeker |
---|
comment:2 Changed 15 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:4 Changed 14 years ago by
Milestone: | → 1.5 |
---|
This should be fixed.