diff options
author | Brandon Aaron <brandon.aaron@gmail.com> | 2007-05-11 22:29:16 +0000 |
---|---|---|
committer | Brandon Aaron <brandon.aaron@gmail.com> | 2007-05-11 22:29:16 +0000 |
commit | 009033bb4566079c1f5006ca0e4521ad64117cd0 (patch) | |
tree | a4c5578cdeac7796e65210e22afadcc742dcf6ee /src | |
parent | f5a2db89fddf65f753b50ff7e77eab47db3b1f23 (diff) | |
download | jquery-009033bb4566079c1f5006ca0e4521ad64117cd0.tar.gz jquery-009033bb4566079c1f5006ca0e4521ad64117cd0.zip |
Fix for #1167
Diffstat (limited to 'src')
-rw-r--r-- | src/jquery/coreTest.js | 7 | ||||
-rw-r--r-- | src/jquery/jquery.js | 2 |
2 files changed, 7 insertions, 2 deletions
diff --git a/src/jquery/coreTest.js b/src/jquery/coreTest.js index d829d3092..dc9c8ed48 100644 --- a/src/jquery/coreTest.js +++ b/src/jquery/coreTest.js @@ -766,7 +766,7 @@ test("addClass(String)", function() { }); test("removeClass(String) - simple", function() { - expect(2); + expect(3); var div = $("div").addClass("test").removeClass("test"), pass = true; for ( var i = 0; i < div.size(); i++ ) { @@ -782,6 +782,11 @@ test("removeClass(String) - simple", function() { if ( div.get(i).className.match(/test|bar|foo/) ) pass = false; } ok( pass, "Remove multiple classes" ); + + reset(); + var div = $("div:eq(0)").addClass("test").removeClass(""); + ok( div.is('.test'), "Empty string passed to removeClass" ); + }); test("toggleClass(String)", function() { diff --git a/src/jquery/jquery.js b/src/jquery/jquery.js index 118b41487..6fb1d5d1d 100644 --- a/src/jquery/jquery.js +++ b/src/jquery/jquery.js @@ -1349,7 +1349,7 @@ jQuery.extend({ // internal only, use removeClass("class") remove: function( elem, c ){ - elem.className = c ? + elem.className = c != undefined ? jQuery.grep( elem.className.split(/\s+/), function(cur){ return !jQuery.className.has( c, cur ); }).join(" ") : ""; |