library-mirrors/jquery-ui
1
0
Fork 0
mirror of https://github.com/jquery/jquery-ui.git synced 2024-11-21 11:04:24 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Effects Core: make removeClass work correctly with changes jQuery 1.9.0. Fixed #9015 - Inclusion of jQuery UI breaks removeClass

Browse Source
This commit is contained in:
David Petersen 2013-01-26 11:59:08 -06:00
parent 3aa8ad0c4a
commit 2ca4d17982
2 changed files with 13 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
tests/unit/effects/effects_core.js
View File

@ -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 /* TODO: Disabled - Can't figure out why this is failing in IE 6/7
test( "createWrapper and removeWrapper retain focused elements (#7595)", function() { test( "createWrapper and removeWrapper retain focused elements (#7595)", function() {

4
ui/jquery.ui.effect.js vendored
View File

@ -849,10 +849,10 @@ $.fn.extend({
_removeClass: $.fn.removeClass, _removeClass: $.fn.removeClass,
removeClass: function( classNames, speed, easing, callback ) { removeClass: function( classNames, speed, easing, callback ) {
return speed ? return arguments.length > 1 ?
$.effects.animateClass.call( this, $.effects.animateClass.call( this,
{ remove: classNames }, speed, easing, callback ) : { remove: classNames }, speed, easing, callback ) :
this._removeClass( classNames ); this._removeClass.apply( this, arguments );
}, },
_toggleClass: $.fn.toggleClass, _toggleClass: $.fn.toggleClass,