Search and Top Navigation
#2024 closed bug (fixed)
Opened December 07, 2007 01:22AM UTC
Closed December 08, 2007 09:56PM UTC
Last modified February 26, 2009 11:20AM UTC
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" });
Fixed in [4086].