Fixes bug where child elements all being false still includes object key in className

Author: developerdizzle

package.json

@@ -1,7 +1,7 @@
 {
   "name": "classcat",
   "description": "0.3 KB JavaScript function for conditionally concatenating CSS class names.",
-  "version": "1.1.3",
+  "version": "2.0.0",
   "main": "dist/classcat.js",
   "jsnext:main": "src/index.js",
   "module": "src/index.js",

src/index.js

@@ -3,25 +3,26 @@ export default function cc(classes, prefix) {
   var className = ""
   var type = typeof classes
 
+  prefix = prefix || ""
+
+  if (type === "boolean") {
+    return classes ? prefix : ""
+  }
+
   if ((classes && type === "string") || type === "number") {
     return classes
   }
 
-  prefix = prefix || " "
-
   if (Array.isArray(classes) && classes.length) {
     for (var i = 0, len = classes.length; i < len; i++) {
       if ((value = cc(classes[i], prefix))) {
-        className += (className && prefix) + value
+        className += (className && " ") + prefix + value
       }
     }
   } else {
     for (var i in classes) {
       if (classes.hasOwnProperty(i) && (value = classes[i])) {
-        className +=
-          (className && prefix) +
-          i +
-          (typeof value === "object" ? cc(value, prefix + i) : "")
+        className += (className && " ") + cc(value, prefix + i)
       }
     }
   }

tests/index.test.js

@@ -43,9 +43,9 @@ test("prefix", () => {
         quux: false,
         baz: true
       },
-      "-"
+      "prefix-"
     )
-  ).toBe("foo-bar-baz")
+  ).toBe("prefix-foo prefix-bar prefix-baz")
 })
 
 test("deep", () => {
@@ -72,3 +72,15 @@ test("not owned props", () => {
 
   delete Object.prototype.myFunction
 })
+
+test("all child elements false", () => {
+  expect(
+    cc([
+      {
+        foo: {
+          "--bar": false
+        }
+      }
+    ])
+  ).toBe("")
+})