#8978 closed bug (duplicate)

Opened January 18, 2013 02:17AM UTC

Closed January 26, 2013 07:22PM UTC

Effects: removeClass without parameters does not remove all classes

Reported by: MichaelM Owned by:
Priority: minor Milestone: 1.11.0
Component: ui.effects.* (individual effect) Version: 1.9.2
Keywords: Cc:
Blocked by: Blocking:
Description

Hello,

I'm using jQuery 1.9.0 and UI 1.9.2, and found that the combination of the two causes removeClass() without any parameters to pass an argument length of 1, causing it to not remove any classes. I managed to fix it by changing a line in my jQuery file, but that's just a temporary hot fix for my application.

Example of empty removeClass() with jQuery UI: http://jsfiddle.net/nbyCC/3/

Example of empty removeClass() without jQuery UI: http://jsfiddle.net/nbyCC/4/

The line I modified:

proceed = arguments.length === 0 || typeof value === "string" && value;
                               ^

to

proceed = arguments.length === 1 || typeof value === "string" && value;
                               ^

Though I see this as an ugly fix.

Attachments (0)
Change History (2)

Changed January 18, 2013 02:59AM UTC by tj.vantoll comment:1

component: ui.core → ui.effects.* (individual effect)
milestone: none → 1.11.0
status: new → open
summary: removeClass() without parameters stops working → Effects: removeClass without parameters does not remove all classes

Verified. When jQuery's docs a call to 

removeClass
with no parameters should remove all classes. When jQuery UI is included this does not happen http://jsfiddle.net/tj_vantoll/KyRMY/.

Changed January 26, 2013 07:22PM UTC by tj.vantoll comment:2

resolution: → duplicate
status: open → closed

Duplicate of #9015.