Skip to main content

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" });
Attachments (0)
Change History (3)

Changed December 08, 2007 09:56PM UTC by rdworth comment:1

resolution: → fixed
status: newclosed

Fixed in [4086].

Changed May 24, 2008 03:39AM UTC by comment:2

milestone: 1.2.2

Milestone 1.2.2 deleted

Changed February 26, 2009 11:20AM UTC by paul comment:3

milestone: → 1.5