postcss-ordered-values - npm Package Compare versions

Comparing version 2.0.1 to 2.0.2

CHANGELOG.md

@@ -0,1 +1,7 @@
+# 2.0.2
+
+* Bump postcss-value-parser to `3.0.1` (thanks to @TrySound).
+* Fixes an issue where the module was discarding color codes if a `calc`
+  function was found (thanks to @TrySound).
+
 # 2.0.1
@@ -2,0 +8,0 @@

dist/index.js

@@ -33,56 +33,47 @@ 'use strict';
 
-var normalizeBorder = function normalizeBorder(decl) {
-    if (! ~borderProps.indexOf(decl.prop)) {
-        return;
-    }
+function normalizeBorder(decl) {
     var order = { width: '', style: '', color: '' };
     var border = (0, _postcssValueParser2['default'])(decl.value);
-    border = border.nodes.filter(function (n) {
-        return n.type !== 'space';
-    });
-    if (border.length > 1) {
-        border.forEach(function (node) {
-            var number = (0, _postcssValueParser.unit)(node.value);
-            if (number || ~borderWidths.indexOf(node.value)) {
-                order.width = node.value + ' ';
+    if (border.nodes.length > 2) {
+        border.walk(function (node) {
+            if (node.type === 'word') {
+                if (~borderStyles.indexOf(node.value)) {
+                    order.style = node.value;
+                    return;
+                }
+                if (~borderWidths.indexOf(node.value) || (0, _postcssValueParser.unit)(node.value)) {
+                    order.width = node.value;
+                    return;
+                }
+                order.color = node.value;
                 return;
             }
-            if (~borderStyles.indexOf(node.value)) {
-                order.style = node.value + ' ';
-                return;
-            }
             if (node.type === 'function') {
-                var value = node.nodes.map(function (n) {
-                    return n.value;
-                }).join('');
-                order.color = node.value + '(' + value + ')';
-                return;
+                if (node.value === 'calc') {
+                    order.width = (0, _postcssValueParser.stringify)(node);
+                } else {
+                    order.color = (0, _postcssValueParser.stringify)(node);
+                }
+                return false;
             }
-            order.color = node.value;
         });
-        decl.value = ('' + order.width + order.style + order.color).trim();
+        decl.value = (order.width + ' ' + order.style + ' ' + order.color).trim();
     }
 };
 
-var normalizeFlexFlow = function normalizeFlexFlow(decl) {
-    if (! ~flexFlowProps.indexOf(decl.prop)) {
-        return;
-    }
+function normalizeFlexFlow(decl) {
     var order = { direction: '', wrap: '' };
     var flexFlow = (0, _postcssValueParser2['default'])(decl.value);
-    flexFlow = flexFlow.nodes.filter(function (n) {
-        return n.type !== 'space';
-    });
-    if (flexFlow.length > 1) {
-        flexFlow.forEach(function (node) {
+    if (flexFlow.nodes.length > 2) {
+        flexFlow.walk(function (node) {
             if (~flexDirection.indexOf(node.value)) {
-                order.direction = node.value + ' ';
+                order.direction = node.value;
                 return;
             }
             if (~flexWrap.indexOf(node.value)) {
-                order.wrap = node.value + ' ';
+                order.wrap = node.value;
                 return;
             }
         });
-        decl.value = ('' + order.direction + order.wrap).trim();
+        decl.value = (order.direction + ' ' + order.wrap).trim();
     }
@@ -94,4 +85,10 @@ };
         css.walkDecls(function (decl) {
-            normalizeBorder(decl);
-            normalizeFlexFlow(decl);
+            if (~borderProps.indexOf(decl.prop)) {
+                normalizeBorder(decl);
+                return;
+            }
+            if (~flexFlowProps.indexOf(decl.prop)) {
+                normalizeFlexFlow(decl);
+                return;
+            }
         });
@@ -98,0 +95,0 @@ };

package.json

 {
   "name": "postcss-ordered-values",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "description": "Ensure values are ordered consistently in your CSS.",
@@ -35,4 +35,4 @@ "main": "dist/index.js",
     "postcss": "^5.0.4",
-    "postcss-value-parser": "^2.0.2"
+    "postcss-value-parser": "^3.0.1"
   }
 }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

7466

increased by1.92%

Worsened metrics

Lines of code

79

decreased by-3.66%

Dependency changes

• + Addedpostcss-value-parser@3.3.1(transitive)

• - Removedpostcss-value-parser@2.0.5(transitive)

• Updatedpostcss-value-parser@^3.0.1