Search and Top Navigation
#9015 closed bug (fixed)
Opened January 26, 2013 03:46PM UTC
Closed January 29, 2013 01:50PM UTC
Last modified January 29, 2013 02:10PM UTC
Inclusion of jQuery UI breaks removeClass
| Reported by: | jbabey | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | 1.10.1 |
| Component: | ui.effects.core | Version: | 1.9.2 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
It appears that something in jQuery UI is conflicting with the removeClass core function, the behavior has been isolated here: http://jsfiddle.net/MvvmJ/9/
Before I suspected it was a bug (which is very rare), I also opened a related stack overflow question here: http://stackoverflow.com/questions/14532327/odd-issue-with-jquery-removeclass-not-doing-anything
If you uncheck jQuery UI in the left "Choose Framework" panel, the final output is as expected: "after blanking: blank" meaning it removed all classes and then added one class ("blank").
With jQuery UI the "destroyed" class is never removed: "after blanking: destroyed blank".
This appears to be happening in both jQuery UI 1.9.2 as well as 1.10.0, with jQuery version 1.9.0 as well as 2.0.0b. I have tested with jQuery 1.6.4, 1.7.2, and 1.8.3 and it does not happen with these versions. I am running my testing in Chrome V24.0.1312.56m with Windows 7 Professional.
Attachments (0)
Change History (7)
Changed January 26, 2013 05:38PM UTC by comment:1
| status: | new → open |
|---|
Changed January 26, 2013 05:55PM UTC by comment:2
This is caused because of this ticket in jQuery and matching commit
Changed January 26, 2013 05:59PM UTC by comment:3
Changed January 29, 2013 01:50PM UTC by comment:5
| resolution: | → fixed |
|---|---|
| status: | open → closed |
Effects Core: make removeClass work correctly with changes jQuery 1.9.0. Fixed #9015 - Inclusion of jQuery UI breaks removeClass
Changeset: 2ca4d17982bfedc723f499b41e0936ada9eaa852
Changed January 29, 2013 02:10PM UTC by comment:6
| milestone: | none → 1.10.1 |
|---|
Changed January 29, 2013 02:10PM UTC by comment:7
| component: | ui.core → ui.effects.core |
|---|