#2024 closed bug (fixed)
Selecting all sliders with ':slider' doesn't work
Reported by: | mistermuckle | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 1.5 |
Component: | ui.core | Version: | 1.2.1 |
Keywords: | slider | Cc: | |
Blocked by: | Blocking: |
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" });
Change History (3)
comment:1 Changed 15 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:3 Changed 14 years ago by
Milestone: | → 1.5 |
---|
Note: See
TracTickets for help on using
tickets.
Fixed in [4086].