Side navigation
#2024 closed bug (fixed)
Opened December 07, 2007 01:22AM UTC
Closed December 08, 2007 09:56PM UTC
Selecting all sliders with ':slider' doesn't work
| Reported by: | mistermuckle | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | 1.2.2 |
| Component: | ui | 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" });
Attachments (0)
Change History (1)
Changed December 08, 2007 09:56PM UTC by comment:1
| resolution: | → fixed |
|---|---|
| status: | new → closed |
Fixed in [4086].