postcss-minify-gradients 4.0.1 → 4.0.2

package/dist/index.js

@@ -30,80 +30,109 @@ const angles = {
 };
 
 function isLessThan(a, b) {
-    return a.unit === b.unit && parseFloat(a.number) >= parseFloat(b.number);
+    return a.unit.toLowerCase() === b.unit.toLowerCase() && parseFloat(a.number) >= parseFloat(b.number);
 }
 
 function optimise(decl) {
-    if (!~decl.value.toLowerCase().indexOf('gradient')) {
+    const value = decl.value;
+
+    if (!~value.toLowerCase().indexOf('gradient')) {
         return;
     }
-    decl.value = (0, _postcssValueParser2.default)(decl.value).walk(node => {
+
+    decl.value = (0, _postcssValueParser2.default)(value).walk(node => {
         if (node.type !== 'function' || !node.nodes.length) {
             return false;
         }
+
         const lowerCasedValue = node.value.toLowerCase();
+
         if (lowerCasedValue === 'linear-gradient' || lowerCasedValue === 'repeating-linear-gradient' || lowerCasedValue === '-webkit-linear-gradient' || lowerCasedValue === '-webkit-repeating-linear-gradient') {
             let args = (0, _cssnanoUtilGetArguments2.default)(node);
+
             if (node.nodes[0].value.toLowerCase() === 'to' && args[0].length === 3) {
                 node.nodes = node.nodes.slice(2);
                 node.nodes[0].value = angles[node.nodes[0].value.toLowerCase()];
             }
+
             let lastStop = null;
+
             args.forEach((arg, index) => {
                 if (!arg[2]) {
                     return;
                 }
+
                 let isFinalStop = index === args.length - 1;
                 let thisStop = (0, _postcssValueParser.unit)(arg[2].value);
+
                 if (lastStop === null) {
                     lastStop = thisStop;
+
                     if (!isFinalStop && lastStop && lastStop.number === '0' && lastStop.unit.toLowerCase() !== 'deg') {
                         arg[1].value = arg[2].value = '';
                     }
+
                     return;
                 }
-                if (isLessThan(lastStop, thisStop)) {
+
+                if (lastStop && thisStop && isLessThan(lastStop, thisStop)) {
                     arg[2].value = 0;
                 }
+
                 lastStop = thisStop;
+
                 if (isFinalStop && arg[2].value === '100%') {
                     arg[1].value = arg[2].value = '';
                 }
             });
+
             return false;
         }
+
         if (lowerCasedValue === 'radial-gradient' || lowerCasedValue === 'repeating-radial-gradient') {
             let args = (0, _cssnanoUtilGetArguments2.default)(node);
             let lastStop;
+
             const hasAt = args[0].find(n => n.value.toLowerCase() === 'at');
+
             args.forEach((arg, index) => {
                 if (!arg[2] || !index && hasAt) {
                     return;
                 }
+
                 let thisStop = (0, _postcssValueParser.unit)(arg[2].value);
+
                 if (!lastStop) {
                     lastStop = thisStop;
+
                     return;
                 }
-                if (isLessThan(lastStop, thisStop)) {
+
+                if (lastStop && thisStop && isLessThan(lastStop, thisStop)) {
                     arg[2].value = 0;
                 }
+
                 lastStop = thisStop;
             });
+
             return false;
         }
+
         if (lowerCasedValue === '-webkit-radial-gradient' || lowerCasedValue === '-webkit-repeating-radial-gradient') {
             let args = (0, _cssnanoUtilGetArguments2.default)(node);
             let lastStop;
+
             args.forEach(arg => {
                 let color;
                 let stop;
+
                 if (arg[2] !== undefined) {
                     if (arg[0].type === 'function') {
                         color = `${arg[0].value}(${(0, _postcssValueParser.stringify)(arg[0].nodes)})`;
                     } else {
                         color = arg[0].value;
                     }
+
                     if (arg[2].type === 'function') {
                         stop = `${arg[2].value}(${(0, _postcssValueParser.stringify)(arg[2].nodes)})`;
                     } else {
@@ -113,23 +142,33 @@ function optimise(decl) {
                     if (arg[0].type === 'function') {
                         color = `${arg[0].value}(${(0, _postcssValueParser.stringify)(arg[0].nodes)})`;
                     }
+
                     color = arg[0].value;
                 }
-                const colorStop = stop || stop === 0 ? (0, _isColorStop2.default)(color, stop) : (0, _isColorStop2.default)(color);
+
+                color = color.toLowerCase();
+
+                const colorStop = stop || stop === 0 ? (0, _isColorStop2.default)(color, stop.toLowerCase()) : (0, _isColorStop2.default)(color);
 
                 if (!colorStop || !arg[2]) {
                     return;
                 }
+
                 let thisStop = (0, _postcssValueParser.unit)(arg[2].value);
+
                 if (!lastStop) {
                     lastStop = thisStop;
+
                     return;
                 }
-                if (isLessThan(lastStop, thisStop)) {
+
+                if (lastStop && thisStop && isLessThan(lastStop, thisStop)) {
                     arg[2].value = 0;
                 }
+
                 lastStop = thisStop;
             });
+
             return false;
         }
     }).toString();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "postcss-minify-gradients",
-  "version": "4.0.1",
+  "version": "4.0.2",
   "description": "Minify gradient parameters with PostCSS.",
   "main": "dist/index.js",
   "scripts": {

package/CHANGELOG.md

@@ -1,34 +0,0 @@
-# 4.0.0-rc.0
-
-* Breaking: Drops support for Node 0.12, we now require at least Node 4.
-* Breaking: Update PostCSS to 6.0.0.
-* Better handling of color stop positions in radial gradients, the module now
-  correctly accounts for the first set of parameters potentially being
-  directional and not stop values.
-
-# 1.0.5
-
-* Resolves a crash on empty `linear-gradient()` functions.
-
-# 1.0.4
-
-* Resolves an issue where `0` was being removed from intermediate colour stops
-  if preceding colour stops contained `calc()` (thanks to @jaybekster).
-
-# 1.0.3
-
-* Resolves an issue where `0` was being incorrectly stripped from the final
-  colour stop value.
-
-# 1.0.2
-
-* Resolves an issue where the module would incorrectly parse floating
-  point numbers.
-
-# 1.0.1
-
-* Reduce function iterations (thanks to @TrySound).
-
-# 1.0.0
-
-* Initial release.