Skip to main content

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 paul comment:1

owner: paulbraeker

Changed May 04, 2008 08:37AM UTC by paul comment:2

resolution: → fixed
status: newclosed

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 paul comment:4

milestone: → 1.5