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].