Changeset 1887

Timestamp:

05/11/07 22:29:16 (2 years ago)

Author:

brandon.aaron

Message:

Fix for #1167

Location:

trunk/jquery/src/jquery

Files:

• coreTest.js (modified) (2 diffs)
• jquery.js (modified) (1 diff)

trunk/jquery/src/jquery/coreTest.js

@@ -767 +767 @@
 
 test("removeClass(String) - simple", function() {
-    expect(2);
+    expect(3);
     var div = $("div").addClass("test").removeClass("test"),
         pass = true;
@@ -783 +783 @@
     }
     ok( pass, "Remove multiple classes" );
+    
+    reset();
+    var div = $("div:eq(0)").addClass("test").removeClass("");
+    ok( div.is('.test'), "Empty string passed to removeClass" );
+    
 });
 

trunk/jquery/src/jquery/jquery.js

@@ -1350 +1350 @@
         // 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 );