Added: JS defined variables are now resolved too (3.2.0)

CHANGELOG.md

@@ -1,3 +1,7 @@
+# 3.2.0 - 2015-03-31
+
+- Added: JS defined variables are now resolved too ([#22](https://github.com/postcss/postcss-custom-properties/issues/22))
+
 # 3.1.0 - 2015-03-16
 
 - Added: variables defined in JS are now automatically prefixed with `--`

index.js

@@ -155,7 +155,7 @@ module.exports = function(options) {
 
     // apply js-defined custom properties
     Object.keys(variables).forEach(function(variable) {
-      map[variable] = variables[variable]
+      map[variable] = resolveValue(variables[variable], map)
     })
 
     // resolve variables

package.json

@@ -1,6 +1,6 @@
 {
   "name": "postcss-custom-properties",
-  "version": "3.1.0",
+  "version": "3.2.0",
   "description": "PostCSS plugin to polyfill W3C CSS Custom Properties for cascading variables",
   "keywords": [
     "css",

test/fixtures/js-defined.css

@@ -4,7 +4,9 @@
 }
 
 div {
-  color: var(--test-one);
-  color: var(--test-two);
-  color: var(--test-three);
+  p: var(--test-one);
+  p: var(--test-two);
+  p: var(--test-three);
+  p: var(--test-varception);
+  p: var(--test-jsception);
 }

test/fixtures/js-defined.expected.css

@@ -1,5 +1,7 @@
 div {
-  color: js-one;
-  color: js-two;
-  color: js-three;
+  p: js-one;
+  p: js-two;
+  p: js-three;
+  p: js-one;
+  p: js-one;
 }

test/index.js

@@ -51,6 +51,8 @@ test("accepts variables defined from JavaScript, and overrides local definitions
       "--test-one": "js-one",
       "--test-two": "js-two",
       "--test-three": "js-three",
+      "--test-varception": "var(--test-one)",
+      "--test-jsception": "var(--test-varception)",
     },
   })
   t.end()