Search and Top Navigation
#2617 closed bug (fixed)
Opened March 29, 2008 06:26PM UTC
Closed May 04, 2008 08:37AM UTC
Last modified February 26, 2009 12:03PM UTC
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.
Attachments (0)
Change History (4)
Changed April 01, 2008 06:47PM UTC by comment:1
owner: | paul → braeker |
---|
Changed May 04, 2008 08:37AM UTC by comment:2
resolution: | → fixed |
---|---|
status: | new → closed |
This should be fixed.
Changed May 24, 2008 03:39AM UTC by comment:3
milestone: | 1.2.4 |
---|
Milestone 1.2.4 deleted
Changed February 26, 2009 12:03PM UTC by comment:4
milestone: | → 1.5 |
---|