postcss-minify-gradients 1.0.1 → 1.0.5

package/CHANGELOG.md

@@ -1,3 +1,22 @@
+# 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).

package/README.md

@@ -41,10 +41,17 @@ See the [PostCSS documentation](https://github.com/postcss/postcss#usage) for
 examples for your environment.
 
 
-## Contributing
+## Contributors
 
-Pull requests are welcome. If you add functionality, then please add unit tests
-to cover it.
+Thanks goes to these wonderful people ([emoji key](https://github.com/kentcdodds/all-contributors#emoji-key)):
+
+<!-- ALL-CONTRIBUTORS-LIST:START - Do not remove or modify this section -->
+| [<img src="https://avatars.githubusercontent.com/u/1282980?v=3" width="100px;"/><br /><sub>Ben Briggs</sub>](http://beneb.info)<br />[💻](https://github.com/ben-eb/postcss-minify-gradients/commits?author=ben-eb) [📖](https://github.com/ben-eb/postcss-minify-gradients/commits?author=ben-eb) 👀 [⚠️](https://github.com/ben-eb/postcss-minify-gradients/commits?author=ben-eb) | [<img src="https://avatars.githubusercontent.com/u/5635476?v=3" width="100px;"/><br /><sub>Bogdan Chadkin</sub>](https://github.com/TrySound)<br />[💻](https://github.com/ben-eb/postcss-minify-gradients/commits?author=TrySound) | [<img src="https://avatars.githubusercontent.com/u/1448788?v=3" width="100px;"/><br /><sub></sub>](https://github.com/huan086)<br />[🐛](https://github.com/ben-eb/postcss-minify-gradients/issues?q=author%3Ahuan086) | [<img src="https://avatars.githubusercontent.com/u/2485494?v=3" width="100px;"/><br /><sub>Mikhail</sub>](https://github.com/jaybekster)<br />[🐛](https://github.com/ben-eb/postcss-minify-gradients/issues?q=author%3Ajaybekster) [💻](https://github.com/ben-eb/postcss-minify-gradients/commits?author=jaybekster) [⚠️](https://github.com/ben-eb/postcss-minify-gradients/commits?author=jaybekster) |
+| :---: | :---: | :---: | :---: |
+<!-- ALL-CONTRIBUTORS-LIST:END -->
+
+This project follows the [all-contributors] specification. Contributions of
+any kind welcome!
 
 
 ## License
@@ -52,6 +59,7 @@ to cover it.
 MIT © [Ben Briggs](http://beneb.info)
 
 
+[all-contributors]: https://github.com/kentcdodds/all-contributors
 [ci]:      https://travis-ci.org/ben-eb/postcss-minify-gradients
 [deps]:    https://gemnasium.com/ben-eb/postcss-minify-gradients
 [npm]:     http://badge.fury.io/js/postcss-minify-gradients

package/dist/index.js

@@ -1,10 +1,8 @@
 'use strict';
 
-Object.defineProperty(exports, '__esModule', {
-    value: true
-});
+exports.__esModule = true;
 
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { 'default': obj }; }
+var _typeof = typeof Symbol === "function" && typeof Symbol.iterator === "symbol" ? function (obj) { return typeof obj; } : function (obj) { return obj && typeof Symbol === "function" && obj.constructor === Symbol ? "symbol" : typeof obj; };
 
 var _postcss = require('postcss');
 
@@ -14,6 +12,8 @@ var _postcssValueParser = require('postcss-value-parser');
 
 var _postcssValueParser2 = _interopRequireDefault(_postcssValueParser);
 
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+
 var angles = {
     top: '0deg',
     right: '90deg',
@@ -33,33 +33,39 @@ function getArguments(node) {
 }
 
 function isLessThan(a, b) {
-    return a.unit === b.unit && parseInt(a.number, 10) >= parseInt(b.number, 10);
+    return a.unit === b.unit && parseFloat(a.number) >= parseFloat(b.number);
 }
 
 function optimise(decl) {
-    if (! ~decl.value.indexOf('gradient')) {
+    if (!~decl.value.indexOf('gradient')) {
         return;
     }
-    decl.value = (0, _postcssValueParser2['default'])(decl.value).walk(function (node) {
+    decl.value = (0, _postcssValueParser2.default)(decl.value).walk(function (node) {
         if (node.type !== 'function') {
             return false;
         }
         if (node.value === 'linear-gradient' || node.value === 'repeating-linear-gradient' || node.value === '-webkit-linear-gradient' || node.value === '-webkit-repeating-linear-gradient') {
-            var _ret = (function () {
+            var _ret = function () {
+                if (!node.nodes.length) {
+                    return {
+                        v: false
+                    };
+                }
                 var args = getArguments(node);
                 if (node.nodes[0].value === 'to' && args[0].length === 3) {
                     node.nodes = node.nodes.slice(2);
                     node.nodes[0].value = angles[node.nodes[0].value];
                 }
-                var lastStop = undefined;
+                var lastStop = null;
                 args.forEach(function (arg, index) {
                     if (!arg[2]) {
                         return;
                     }
+                    var isFinalStop = index === args.length - 1;
                     var thisStop = (0, _postcssValueParser.unit)(arg[2].value);
-                    if (!lastStop) {
+                    if (lastStop === null) {
                         lastStop = thisStop;
-                        if (lastStop && lastStop.number === '0' && lastStop.unit !== 'deg') {
+                        if (!isFinalStop && lastStop && lastStop.number === '0' && lastStop.unit !== 'deg') {
                             arg[1].value = arg[2].value = '';
                         }
                         return;
@@ -68,21 +74,26 @@ function optimise(decl) {
                         arg[2].value = 0;
                     }
                     lastStop = thisStop;
-                    if (index === args.length - 1 && arg[2].value === '100%') {
+                    if (isFinalStop && arg[2].value === '100%') {
                         arg[1].value = arg[2].value = '';
                     }
                 });
                 return {
                     v: false
                 };
-            })();
+            }();
 
-            if (typeof _ret === 'object') return _ret.v;
+            if ((typeof _ret === 'undefined' ? 'undefined' : _typeof(_ret)) === "object") return _ret.v;
         }
         if (node.value === 'radial-gradient' || node.value === 'repeating-radial-gradient' || node.value === '-webkit-radial-gradient' || node.value === '-webkit-repeating-radial-gradient') {
-            var _ret2 = (function () {
+            var _ret2 = function () {
+                if (!node.nodes.length) {
+                    return {
+                        v: false
+                    };
+                }
                 var args = getArguments(node);
-                var lastStop = undefined;
+                var lastStop = void 0;
                 args.forEach(function (arg) {
                     if (!arg[2]) {
                         return;
@@ -100,14 +111,14 @@ function optimise(decl) {
                 return {
                     v: false
                 };
-            })();
+            }();
 
-            if (typeof _ret2 === 'object') return _ret2.v;
+            if ((typeof _ret2 === 'undefined' ? 'undefined' : _typeof(_ret2)) === "object") return _ret2.v;
         }
     }).toString();
 }
 
-exports['default'] = _postcss2['default'].plugin('postcss-minify-gradients', function () {
+exports.default = _postcss2.default.plugin('postcss-minify-gradients', function () {
     return function (css) {
         return css.walkDecls(optimise);
     };

package/package.json

@@ -1,12 +1,15 @@
 {
   "name": "postcss-minify-gradients",
-  "version": "1.0.1",
+  "version": "1.0.5",
   "description": "Minify gradient parameters with PostCSS.",
   "main": "dist/index.js",
   "scripts": {
-    "lint": "eslint src",
-    "prepublish": "babel src --out-dir dist --ignore /__tests__/",
-    "test": "npm run lint && babel-tape-runner \"src/**/__tests__/*.js\" | tap-spec"
+    "contributorAdd": "all-contributors add",
+    "contributorGenerate": "all-contributors generate",
+    "pretest": "eslint src",
+    "prepublish": "del-cli dist && BABEL_ENV=publish babel src --out-dir dist --ignore /__tests__/",
+    "test": "ava src/__tests__",
+    "test-012": "ava src/__tests__"
   },
   "files": [
     "LICENSE-MIT",
@@ -19,11 +22,20 @@
   ],
   "license": "MIT",
   "devDependencies": {
-    "babel": "^5.8.23",
-    "babel-tape-runner": "1.2.0",
-    "eslint": "^1.10.3",
-    "tap-spec": "^4.1.1",
-    "tape": "^4.2.2"
+    "all-contributors-cli": "^3.0.5",
+    "ava": "^0.16.0",
+    "babel-cli": "^6.3.17",
+    "babel-core": "^6.3.26",
+    "babel-plugin-add-module-exports": "^0.2.1",
+    "babel-preset-es2015": "^6.3.13",
+    "babel-preset-es2015-loose": "^7.0.0",
+    "babel-preset-stage-0": "^6.3.13",
+    "babel-register": "^6.9.0",
+    "del-cli": "^0.2.0",
+    "eslint": "^3.0.0",
+    "eslint-config-cssnano": "^3.0.0",
+    "eslint-plugin-babel": "^3.3.0",
+    "eslint-plugin-import": "^2.0.1"
   },
   "homepage": "https://github.com/ben-eb/postcss-minify-gradients",
   "author": {
@@ -34,6 +46,12 @@
   "repository": "ben-eb/postcss-minify-gradients",
   "dependencies": {
     "postcss": "^5.0.12",
-    "postcss-value-parser": "^3.1.3"
+    "postcss-value-parser": "^3.3.0"
+  },
+  "ava": {
+    "require": "babel-register"
+  },
+  "eslintConfig": {
+    "extends": "cssnano"
   }
 }