Merge pull request #3 from dollarshaveclub/classname-syntax

Changing the way class names are attached to the body
dollarshaveclub · Sep 20, 2016 · a97b66a · a97b66a
2 parents 63ec134 + 3b9ffbb
commit a97b66a
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 7 deletions.
diff --git a/build/test-interface.min.js b/build/test-interface.min.js
diff --git a/build/test.min.js b/build/test.min.js
diff --git a/lib/index.js b/lib/index.js
@@ -76,10 +76,11 @@ class Test {
 
     // We've now bucketed our user
     // Add classname
+    const className = `${name}--${bucket}`;
     if (supportsClasslist) {
-      document.body.classList.add(name, bucket);
+      document.body.classList.add(className);
     } else {
-      document.body.className += ` ${name} ${bucket}`;
+      document.body.className += ` ${className}`;
     }
 
     let info = {

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "ab-tester",
-  "version": "0.3.2",
+  "version": "0.4.0",
   "homepage": "https://github.com/dollarshaveclub/ab-tester.js",
   "main": "./index.js",
   "repository": {
@@ -28,5 +28,8 @@
     "sinon": "1.15.4",
     "vinyl-buffer": "^1.0.0",
     "vinyl-source-stream": "^1.1.0"
+  },
+  "dependencies": {
+    "babel-preset-es2015": "^6.14.0"
   }
 }