Use LF line endings

Author: MadLittleMods

.gitignore

@@ -3,4 +3,4 @@ npm-debug.log
 
 playground/node_modules/
 playground/jspm_packages/
-todo.md
+todo.md

CHANGELOG.md

@@ -1,111 +1,111 @@
-
-# v0.9.0 - 2018-6-26
-
- - Adds `opts.preserveInjectedVariables`, which when set to `false`, removes the `:root { ... }` custom property declarations added via `opts.variables`
-    - Thank you to [@akdetrick](https://github.com/akdetrick) for the [contribution](https://github.com/MadLittleMods/postcss-css-variables/pull/74)
-
-
-# v0.8.1 - 2018-3-21
-
- - Log `undefined` variables (available in `result.warnings()`)
-    - Thank you to [@pixeldrew](https://github.com/pixeldrew) for the [contribution](https://github.com/MadLittleMods/postcss-css-variables/pull/69)
-
-
-# v0.8.0 - 2017-8-8
-
- - Remove PostCSS `moveTo`/`append` deprecation warnings, [#50](https://github.com/MadLittleMods/postcss-css-variables/issues/50)
-    - Thank you to [@modosc](https://github.com/modosc) for the [contribution](https://github.com/MadLittleMods/postcss-css-variables/pull/56)
-
-
-# v0.7.0 - 2017-3-12
-
- - Resolve `var` usage in fallbacks, [#37](https://github.com/MadLittleMods/postcss-css-variables/issues/37)
-    - Thank you to [@asvny](https://github.com/asvny) and [@marklu](https://github.com/marklu) for the contribution, [#39](https://github.com/MadLittleMods/postcss-css-variables/issues/39) -> [#49](https://github.com/MadLittleMods/postcss-css-variables/pull/49)
-
-
-# v0.6.0 - 2016-9-23
-
- - Update/refactor readme
-    - Thank you to [@isiahmeadows](github.com/isiahmeadows) for the [contribution](https://github.com/MadLittleMods/postcss-css-variables/pull/36)
- - Use string value for `undefined` variables to play nice with other plugins downstream
-    - Thank you to [@vincentorback](github.com/vincentorback) for the [contribution](https://github.com/MadLittleMods/postcss-css-variables/pull/44)
-
-
-# v0.5.2 - 2016-8-24
-
- - Fix [#42](https://github.com/MadLittleMods/postcss-css-variables/issues/42) where `opts.preserve` was not working inside at-rules
-    - Thank you to [@muftiev](github.com/muftiev) for the [contribution](https://github.com/MadLittleMods/postcss-css-variables/pull/43)
-
-
-# v0.5.1 - 2015-10-24
-
- - Fix [postcss/postcss#611](https://github.com/postcss/postcss/issues/611) where we were trying to remove the root node on clean up
- - Improved test setup
-
-# v0.5.0 - 2015-9-12
-
- - Upgrade to PostCSS v5. Fix [#20](https://github.com/MadLittleMods/postcss-css-variables/issues/20)
-
-# v0.4.0 - 2015-7-2
-
- - Fix [#15](https://github.com/MadLittleMods/postcss-css-variables/issues/15)
- 	 - Remove slowness from cloning the `root` with `node.clone().removeAll()`. Now using `./lib/shallow-clone-node.js` to avoid cloning children which will get removed right after.
- 	 - Thank you to [@ddprrt](https://github.com/ddprrt) for bringing up the slowness issue in this article, [PostCSS misconceptions](https://medium.com/@ddprrt/postcss-misconceptions-faf5dc5038df).
-
-
-
-# v0.3.9 - 2015-6-29
-
- - Remove `opts` global leak. Fix [#13](https://github.com/MadLittleMods/postcss-css-variables/issues/13)
-
-
-# v0.3.8 - 2015-5-28
-
- - Add support for pseudo selectors `:hover` `:before`
-
-# v0.3.7 - 2015-5-27
-
- - Fix [#7](https://github.com/MadLittleMods/postcss-css-variables/issues/7): Support for child combinator
- - Added tests for child-combinator/direct-descendant coverage
-
-# v0.3.6 - 2015-5-21
-
- - Fix [#6](https://github.com/MadLittleMods/postcss-css-variables/issues/6). Variable usage in comma separated selector to use proper scope
-
-# v0.3.5 - 2015-5-12
-
- - Big refactor of code to reduce cyclomatic complexity. Still needs work though.
- - Fix variable referencing another variable resolution when being changed by at-rule in non-root rule
-
-# v0.3.4 - 2015-5-12
-
- - Fix variable referencing another variable resolution when being changed by at-rule
-
-# v0.3.3 - 2015-5-11
-
- - Add support for last piece of combinator chain in selector resolution matching.
- 	 - `.foo + .bar` can match variables declared in `.bar`
-
-# v0.3.1 - 2015-5-5
-
- - Large overhaul of code to make it more robust on proper scope resolution.
- - Fix [#2]](https://github.com/MadLittleMods/postcss-css-variables/issues/2)
-
-# v0.2.3 - 2015-5-4
-
- - Add support for CSS4 descendant selector `>>` syntax
-
-# v0.2.2 - 2015-5-1
-
- - Automatically prefix any variables defined in `options.variables` with `--` (according to CSS custom property syntax).
-
-# v0.2.1 - 2015-4-30
-
- - Added support for descendant selector nesting instead of just physical space nesting
- - Fixed issue with comma separated rules. It was throwing a undefined is not a function error
- - Moved to external scope check `isUnderScope` instead of integrated into `resolveValue`
- - Added test for empty `var()` call. See [test/fixtures/empty-var-func.css](https://github.com/MadLittleMods/postcss-css-variables/blob/master/test/fixtures/empty-var-func.css)
-
-# v0.1.0 - 2015-4-29
-
- - First release
+
+# v0.9.0 - 2018-6-26
+
+ - Adds `opts.preserveInjectedVariables`, which when set to `false`, removes the `:root { ... }` custom property declarations added via `opts.variables`
+    - Thank you to [@akdetrick](https://github.com/akdetrick) for the [contribution](https://github.com/MadLittleMods/postcss-css-variables/pull/74)
+
+
+# v0.8.1 - 2018-3-21
+
+ - Log `undefined` variables (available in `result.warnings()`)
+    - Thank you to [@pixeldrew](https://github.com/pixeldrew) for the [contribution](https://github.com/MadLittleMods/postcss-css-variables/pull/69)
+
+
+# v0.8.0 - 2017-8-8
+
+ - Remove PostCSS `moveTo`/`append` deprecation warnings, [#50](https://github.com/MadLittleMods/postcss-css-variables/issues/50)
+    - Thank you to [@modosc](https://github.com/modosc) for the [contribution](https://github.com/MadLittleMods/postcss-css-variables/pull/56)
+
+
+# v0.7.0 - 2017-3-12
+
+ - Resolve `var` usage in fallbacks, [#37](https://github.com/MadLittleMods/postcss-css-variables/issues/37)
+    - Thank you to [@asvny](https://github.com/asvny) and [@marklu](https://github.com/marklu) for the contribution, [#39](https://github.com/MadLittleMods/postcss-css-variables/issues/39) -> [#49](https://github.com/MadLittleMods/postcss-css-variables/pull/49)
+
+
+# v0.6.0 - 2016-9-23
+
+ - Update/refactor readme
+    - Thank you to [@isiahmeadows](github.com/isiahmeadows) for the [contribution](https://github.com/MadLittleMods/postcss-css-variables/pull/36)
+ - Use string value for `undefined` variables to play nice with other plugins downstream
+    - Thank you to [@vincentorback](github.com/vincentorback) for the [contribution](https://github.com/MadLittleMods/postcss-css-variables/pull/44)
+
+
+# v0.5.2 - 2016-8-24
+
+ - Fix [#42](https://github.com/MadLittleMods/postcss-css-variables/issues/42) where `opts.preserve` was not working inside at-rules
+    - Thank you to [@muftiev](github.com/muftiev) for the [contribution](https://github.com/MadLittleMods/postcss-css-variables/pull/43)
+
+
+# v0.5.1 - 2015-10-24
+
+ - Fix [postcss/postcss#611](https://github.com/postcss/postcss/issues/611) where we were trying to remove the root node on clean up
+ - Improved test setup
+
+# v0.5.0 - 2015-9-12
+
+ - Upgrade to PostCSS v5. Fix [#20](https://github.com/MadLittleMods/postcss-css-variables/issues/20)
+
+# v0.4.0 - 2015-7-2
+
+ - Fix [#15](https://github.com/MadLittleMods/postcss-css-variables/issues/15)
+ 	 - Remove slowness from cloning the `root` with `node.clone().removeAll()`. Now using `./lib/shallow-clone-node.js` to avoid cloning children which will get removed right after.
+ 	 - Thank you to [@ddprrt](https://github.com/ddprrt) for bringing up the slowness issue in this article, [PostCSS misconceptions](https://medium.com/@ddprrt/postcss-misconceptions-faf5dc5038df).
+
+
+
+# v0.3.9 - 2015-6-29
+
+ - Remove `opts` global leak. Fix [#13](https://github.com/MadLittleMods/postcss-css-variables/issues/13)
+
+
+# v0.3.8 - 2015-5-28
+
+ - Add support for pseudo selectors `:hover` `:before`
+
+# v0.3.7 - 2015-5-27
+
+ - Fix [#7](https://github.com/MadLittleMods/postcss-css-variables/issues/7): Support for child combinator
+ - Added tests for child-combinator/direct-descendant coverage
+
+# v0.3.6 - 2015-5-21
+
+ - Fix [#6](https://github.com/MadLittleMods/postcss-css-variables/issues/6). Variable usage in comma separated selector to use proper scope
+
+# v0.3.5 - 2015-5-12
+
+ - Big refactor of code to reduce cyclomatic complexity. Still needs work though.
+ - Fix variable referencing another variable resolution when being changed by at-rule in non-root rule
+
+# v0.3.4 - 2015-5-12
+
+ - Fix variable referencing another variable resolution when being changed by at-rule
+
+# v0.3.3 - 2015-5-11
+
+ - Add support for last piece of combinator chain in selector resolution matching.
+ 	 - `.foo + .bar` can match variables declared in `.bar`
+
+# v0.3.1 - 2015-5-5
+
+ - Large overhaul of code to make it more robust on proper scope resolution.
+ - Fix [#2]](https://github.com/MadLittleMods/postcss-css-variables/issues/2)
+
+# v0.2.3 - 2015-5-4
+
+ - Add support for CSS4 descendant selector `>>` syntax
+
+# v0.2.2 - 2015-5-1
+
+ - Automatically prefix any variables defined in `options.variables` with `--` (according to CSS custom property syntax).
+
+# v0.2.1 - 2015-4-30
+
+ - Added support for descendant selector nesting instead of just physical space nesting
+ - Fixed issue with comma separated rules. It was throwing a undefined is not a function error
+ - Moved to external scope check `isUnderScope` instead of integrated into `resolveValue`
+ - Added test for empty `var()` call. See [test/fixtures/empty-var-func.css](https://github.com/MadLittleMods/postcss-css-variables/blob/master/test/fixtures/empty-var-func.css)
+
+# v0.1.0 - 2015-4-29
+
+ - First release

index.js

@@ -151,6 +151,7 @@ module.exports = postcss.plugin('postcss-css-variables', function(options) {
 		// Collect all of the variables defined
 		// ---------------------------------------------------------
 		// ---------------------------------------------------------
+		//console.log('Collecting variables defined START');
 		eachCssVariableDeclaration(css, function(decl) {
 			var declParentRule = decl.parent;
 
@@ -195,6 +196,7 @@ module.exports = postcss.plugin('postcss-css-variables', function(options) {
 				nodesToRemoveAtEnd.push(declParentRule);
 			}
 		});
+		//console.log('Collecting variables defined END');
 
 
 

lib/clone-splice-parent-onto-node-when.js

@@ -1,67 +1,67 @@
-
-var shallowCloneNode = require('./shallow-clone-node');
-
-// Splice on a parent scope onto a node
-// And return a detached clone
-var cloneSpliceParentOntoNodeWhen = function(node, parent, /*optional*/whenCb) {
-	whenCb = whenCb || function() {
-		return true;
-	};
-
-	var cloneList = [];
-
-	// Gather node ancestors and clone along the way
-	var current = node;
-	var isWhenNow = false;
-	while(current && !isWhenNow) {
-		if(current.type === 'decl') {
-			cloneList.push(current.clone());
-		}
-		else {
-			cloneList.push(shallowCloneNode(current));
-		}
-
-		isWhenNow = whenCb(current);
-		current = current.parent;
-	}
-
-
-	// Gather parent ancestors all the way up and clone along the way
-	// The list goes from lowest to highest ancestor
-	var cloneParentList = [];
-	var currentParent = parent;
-	while(currentParent) {
-		cloneParentList.push(shallowCloneNode(currentParent));
-
-		currentParent = currentParent.parent;
-	}
-	// Assign parents to our parent clones
-	cloneParentList.forEach(function(parentClone, index, cloneParentList) {
-		// Keep assigning parents detached until just very end
-		if(index + 1 < cloneParentList.length) {
-			//parentClone.moveTo(cloneParentList[index+1]);
-			parentClone.parent = cloneParentList[index + 1];
-		}
-	});
-
-
-	// Assign parents to our node clones
-	cloneList.forEach(function(clone, index, cloneList) {
-		// Keep assigning parents detached until just very end
-		if(index + 1 < cloneList.length) {
-			//clone.moveTo(cloneList[index+1]);
-			clone.parent = cloneList[index + 1];
-		// Then splice on the new parent scope
-		} else {
-			// Set the highest parent ancestor to back to where we should splice in
-			cloneParentList.slice(-1)[0].parent = current;
-			// Set the node clone to the lowest parent ancestor to finish off the splice
-			//clone.moveTo(cloneParentList[0]);
-			clone.parent = cloneParentList[0];
-		}
-	});
-
-	return cloneList[0];
-};
-
-module.exports = cloneSpliceParentOntoNodeWhen;
+
+var shallowCloneNode = require('./shallow-clone-node');
+
+// Splice on a parent scope onto a node
+// And return a detached clone
+var cloneSpliceParentOntoNodeWhen = function(node, parent, /*optional*/whenCb) {
+	whenCb = whenCb || function() {
+		return true;
+	};
+
+	var cloneList = [];
+
+	// Gather node ancestors and clone along the way
+	var current = node;
+	var isWhenNow = false;
+	while(current && !isWhenNow) {
+		if(current.type === 'decl') {
+			cloneList.push(current.clone());
+		}
+		else {
+			cloneList.push(shallowCloneNode(current));
+		}
+
+		isWhenNow = whenCb(current);
+		current = current.parent;
+	}
+
+
+	// Gather parent ancestors all the way up and clone along the way
+	// The list goes from lowest to highest ancestor
+	var cloneParentList = [];
+	var currentParent = parent;
+	while(currentParent) {
+		cloneParentList.push(shallowCloneNode(currentParent));
+
+		currentParent = currentParent.parent;
+	}
+	// Assign parents to our parent clones
+	cloneParentList.forEach(function(parentClone, index, cloneParentList) {
+		// Keep assigning parents detached until just very end
+		if(index + 1 < cloneParentList.length) {
+			//parentClone.moveTo(cloneParentList[index+1]);
+			parentClone.parent = cloneParentList[index + 1];
+		}
+	});
+
+
+	// Assign parents to our node clones
+	cloneList.forEach(function(clone, index, cloneList) {
+		// Keep assigning parents detached until just very end
+		if(index + 1 < cloneList.length) {
+			//clone.moveTo(cloneList[index+1]);
+			clone.parent = cloneList[index + 1];
+		// Then splice on the new parent scope
+		} else {
+			// Set the highest parent ancestor to back to where we should splice in
+			cloneParentList.slice(-1)[0].parent = current;
+			// Set the node clone to the lowest parent ancestor to finish off the splice
+			//clone.moveTo(cloneParentList[0]);
+			clone.parent = cloneParentList[0];
+		}
+	});
+
+	return cloneList[0];
+};
+
+module.exports = cloneSpliceParentOntoNodeWhen;