Bug Tracker

Opened 12 years ago

Closed 12 years ago

Last modified 8 years ago

#2617 closed bug (fixed)

ui.slider.js range focus bug.

Reported by: norbert Owned by: braeker
Priority: major Milestone: 1.2.4
Component: ui 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 (2)

comment:1 Changed 12 years ago by paul

Owner: changed from paul to braeker

comment:2 Changed 12 years ago by paul

Resolution: fixed
Status: newclosed

This should be fixed.

Note: See TracTickets for help on using tickets.