Bug Tracker

Ticket #2024 (closed bug: fixed)

Opened 1 year ago

Last modified 1 year ago

Selecting all sliders with ':slider' doesn't work

Reported by: mistermuckle Assigned to: anonymous
Type: bug Priority: minor
Milestone: 1.2.2 Component: ui
Version: 1.2.1 Keywords: slider
Cc: Needs: Review

Description

Lines 55-56 of ui.slider.js say:

//Make nodes selectable by expression
$.extend($.expr[':'], { slider: "(' '+a.className+' ').indexOf(' ui-slider ')" });

indexOf() returns -1 on failure, which evaluates to true, and the result is that $(':slider') selects every element on the page. Adding >=0 fixes the problem:

//Make nodes selectable by expression
$.extend($.expr[':'], { slider: "(' '+a.className+' ').indexOf(' ui-slider ')>=0" });

Attachments

Change History

Changed 1 year ago by rworth

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

Fixed in [4086].

Note: See TracTickets for help on using tickets.