Bug Tracker

Ticket #2617 (closed bug: fixed)

Opened 8 months ago

Last modified 7 months ago

ui.slider.js range focus bug.

Reported by: norbert Assigned to: braeker
Type: bug Priority: major
Milestone: 1.2.4 Component: ui
Version: 1.2.3 Keywords:
Cc: Needs: Review

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

Change History

Changed 8 months ago by paul

  • owner changed from paul to braeker

Changed 7 months ago by paul

  • status changed from new to closed
  • resolution set to fixed

This should be fixed.

Note: See TracTickets for help on using tickets.