Fix bug when removing a css class

johnthethird · Mar 17, 2012 · 6ddebae · 6ddebae
1 parent 8bdb947
commit 6ddebae
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/batman.coffee b/src/batman.coffee
@@ -4991,7 +4991,7 @@ class Batman.DOM.AddClassBinding extends Batman.DOM.AbstractAttributeBinding
       if !!value is !@invert
         @node.className = "#{currentName} #{name}" if !includesClassName
       else
-        @node.className = currentName.replace(pattern, '') if includesClassName
+        @node.className = currentName.replace(pattern, ' ') if includesClassName
     true
 
 class Batman.DOM.EventBinding extends Batman.DOM.AbstractAttributeBinding

diff --git a/tests/batman/view/class_binding_test.coffee b/tests/batman/view/class_binding_test.coffee
@@ -51,19 +51,21 @@ asyncTest 'it should allow multiple class names to be bound and updated', ->
       equal node[0].className, 'bar baz'
 
 
-asyncTest 'it should allow an already present class to be removed', 4, ->
-  source = '<div data-removeclass-two="bar" class="zero two"></div>'
+asyncTest 'it should allow an already present class to be removed', 6, ->
+  source = '<div data-removeclass-two="bar" class="zero two three"></div>'
   context = Batman
     foo: true
     bar: false
   helpers.render source, context, (node) ->
     ok node.hasClass('zero')
     ok node.hasClass('two')
+    ok node.hasClass('three')
 
     context.set 'bar', true
     delay ->
       ok node.hasClass('zero')
       ok !node.hasClass('two')
+      ok node.hasClass('three')
 
 asyncTest 'it should not remove an already present similar class name', 6, ->
   source = '<div data-removeclass-foobar="bar" class="zero bar"></div>'