fix: fix errors caused by empty class names getting applied to Surfac…

…es via
Famous · Jul 24, 2014 · 4054c44 · 4054c44
1 parent 5e774ce
commit 4054c44
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/dist/famous-angular.js b/dist/famous-angular.js
@@ -401,9 +401,10 @@ angular.module('famous.angular')
         angular.element.prototype[classManipulator] = function(className) {
           originalManipulator.apply(this, arguments);
 
-          // If and only if the current element represents a Famo.us Surface, pass through
+          // If and only if the current element represents a Famo.us Surface,
+          // AND the class is not an empty string, pass through
           // the addClass and removeClass methods to the underlying renderNode.
-          if (isClassable(this)) {
+          if (isClassable(this) && typeof className === 'string' && className.trim() !== '') {
             $famous.getIsolate(this.scope()).renderNode[classManipulator](className);
           }
         };