fix #76 removeClass

vuejs · Feb 7, 2014 · 4fdeba5 · 4fdeba5
1 parent be74193
commit 4fdeba5
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/src/utils.js b/src/utils.js
@@ -221,9 +221,12 @@ var utils = module.exports = {
         if (hasClassList) {
             el.classList.remove(cls)
         } else {
-            el.className = (' ' + el.className + ' ')
-                .replace(' ' + cls + ' ', '')
-                .trim()
+            var cur = ' ' + el.className + ' ',
+                tar = ' ' + cls + ' '
+            while (cur.indexOf(tar) >= 0) {
+                cur = cur.replace(tar, ' ')
+            }
+            el.className = cur.trim()
         }
     }
 }
diff --git a/test/unit/specs/utils.js b/test/unit/specs/utils.js
@@ -341,8 +341,10 @@ describe('UNIT: Utils', function () {
 
         it('should work', function () {
             var el = document.createElement('div')
-            el.className = 'hihi hi'
+            el.className = 'hihi hi ha'
             utils.removeClass(el, 'hi')
+            assert.strictEqual(el.className, 'hihi ha')
+            utils.removeClass(el, 'ha')
             assert.strictEqual(el.className, 'hihi')
         })