diff options
| author | David Petersen <public@petersendidit.com> | 2013-01-26 11:59:08 -0600 |
|---|---|---|
| committer | David Petersen <public@petersendidit.com> | 2013-01-29 07:47:54 -0600 |
| commit | 2ca4d17982bfedc723f499b41e0936ada9eaa852 (patch) | |
| tree | 97f40e694c062a367ce5e358c911a50367777700 | |
| parent | 3aa8ad0c4ad15ceac52c4f653b7d4e66995bb297 (diff) | |
| download | jquery-ui-2ca4d17982bfedc723f499b41e0936ada9eaa852.tar.gz jquery-ui-2ca4d17982bfedc723f499b41e0936ada9eaa852.zip | |
Effects Core: make removeClass work correctly with changes jQuery 1.9.0. Fixed #9015 - Inclusion of jQuery UI breaks removeClass
| -rw-r--r-- | tests/unit/effects/effects_core.js | 11 | ||||
| -rw-r--r-- | ui/jquery.ui.effect.js | 4 |
2 files changed, 13 insertions, 2 deletions
diff --git a/tests/unit/effects/effects_core.js b/tests/unit/effects/effects_core.js index 26390b7d3..c9b1e1b4a 100644 --- a/tests/unit/effects/effects_core.js +++ b/tests/unit/effects/effects_core.js @@ -38,6 +38,17 @@ asyncTest( "Parse of null for options", function() { }); }); +test( "removeClass", function() { + expect( 3 ); + + var element = $( "<div>" ); + equal( "", element[ 0 ].className ); + element.addClass( "destroyed" ); + equal( "destroyed", element[ 0 ].className ); + element.removeClass(); + equal( "", element[ 0 ].className ); +}); + /* TODO: Disabled - Can't figure out why this is failing in IE 6/7 test( "createWrapper and removeWrapper retain focused elements (#7595)", function() { diff --git a/ui/jquery.ui.effect.js b/ui/jquery.ui.effect.js index c6623a0a1..97f006ee0 100644 --- a/ui/jquery.ui.effect.js +++ b/ui/jquery.ui.effect.js @@ -849,10 +849,10 @@ $.fn.extend({ _removeClass: $.fn.removeClass, removeClass: function( classNames, speed, easing, callback ) { - return speed ? + return arguments.length > 1 ? $.effects.animateClass.call( this, { remove: classNames }, speed, easing, callback ) : - this._removeClass( classNames ); + this._removeClass.apply( this, arguments ); }, _toggleClass: $.fn.toggleClass, |