postcss-minify-params 1.0.5 → 1.2.2

package/CHANGELOG.md

@@ -1,3 +1,23 @@
+# 1.2.2
+
+* Resolves an issue where `all and` would be removed from
+  `@media not all and (conditions) {}`, causing an invalid media query to
+  be output.
+
+# 1.2.1
+
+* Resolves an issue where `1.2.0` would throw on empty function parentheses.
+
+# 1.2.0
+
+* Adds support for simplifying `min-aspect-ratio` and `max-aspect-ratio`. For
+  example, `@media (min-aspect-ratio: 1280/720) {}` can be minified to
+  `@media (min-aspect-ratio:16/9){}`.
+
+# 1.1.0
+
+* Adds support for removing the unnecessary `all and` from media queries.
+
 # 1.0.1
 
 * Refactor to ES6.

package/README.md

@@ -29,5 +29,5 @@ See [PostCSS] docs for examples for your environment.
 MIT © [Bogdan Chadkin](mailto:trysound@yandex.ru)
 
 [PostCSS]: https://github.com/postcss/postcss
-[ci-img]:  https://travis-ci.org/TrySound/postcss-minify-params.svg
-[ci]:      https://travis-ci.org/TrySound/postcss-minify-params
+[ci-img]:  https://travis-ci.org/ben-eb/postcss-minify-params.svg
+[ci]:      https://travis-ci.org/ben-eb/postcss-minify-params

package/dist/index.js

@@ -4,6 +4,21 @@ var stringify = valueParser.stringify;
 var sort = require('alphanum-sort');
 var uniqs = require('uniqs');
 
+/**
+ * Return the greatest common divisor
+ * of two numbers.
+ */
+
+function gcd(a, b) {
+    return b ? gcd(b, a % b) : a;
+}
+
+function aspectRatio(a, b) {
+    var divisor = gcd(a, b);
+
+    return [a / divisor, b / divisor];
+}
+
 function split(nodes, div) {
     var result = [];
     var i, max, node;
@@ -24,6 +39,11 @@ function split(nodes, div) {
     return result;
 }
 
+function removeNode(node) {
+    node.value = '';
+    node.type = 'word';
+}
+
 module.exports = postcss.plugin('postcss-minify-params', function () {
     return function (css) {
         css.walkAtRules(function (rule) {
@@ -33,17 +53,54 @@ module.exports = postcss.plugin('postcss-minify-params', function () {
 
             var params = valueParser(rule.params);
 
-            params.walk(function (node) {
+            params.walk(function (node, index) {
                 if (node.type === 'div' || node.type === 'function') {
                     node.before = node.after = '';
+                    if (
+                        node.type === 'function' &&
+                        node.nodes[4] &&
+                        node.nodes[0].value.indexOf('-aspect-ratio') === 3
+                    ) {
+                        var ref = aspectRatio(
+                            node.nodes[2].value,
+                            node.nodes[4].value
+                        );
+                        var a = ref[0];
+                        var b = ref[1];
+                        node.nodes[2].value = a;
+                        node.nodes[4].value = b;
+                    }
                 } else if (node.type === 'space') {
                     node.value = ' ';
+                } else if (node.type === 'word') {
+                    var prevWord = params.nodes[index - 2];
+                    if (
+                        node.value === 'all' &&
+                        rule.name === 'media' &&
+                        !prevWord
+                    ) {
+                        var nextSpace = params.nodes[index + 1];
+                        var nextWord = params.nodes[index + 2];
+                        var secondSpace = params.nodes[index + 3];
+                        if (nextWord && nextWord.value === 'and') {
+                            removeNode(nextWord);
+                            removeNode(nextSpace);
+                            if (secondSpace) {
+                                removeNode(secondSpace);
+                            }
+                        }
+                        removeNode(node);
+                    }
                 }
             }, true);
 
             rule.params = sort(uniqs(split(params.nodes, ',')), {
                 insensitive: true
             }).join();
+
+            if (!rule.params.length) {
+                rule.raws.afterName = '';
+            }
         });
     };
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "postcss-minify-params",
-  "version": "1.0.5",
+  "version": "1.2.2",
   "description": "Minify at-rule params with PostCSS",
   "keywords": [
     "postcss",
@@ -16,11 +16,11 @@
   ],
   "author": "Bogdan Chadkin <trysound@yandex.ru>",
   "license": "MIT",
-  "repository": "TrySound/postcss-minify-params",
+  "repository": "ben-eb/postcss-minify-params",
   "bugs": {
-    "url": "https://github.com/TrySound/postcss-minify-params/issues"
+    "url": "https://github.com/ben-eb/postcss-minify-params/issues"
   },
-  "homepage": "https://github.com/TrySound/postcss-minify-params",
+  "homepage": "https://github.com/ben-eb/postcss-minify-params",
   "dependencies": {
     "alphanum-sort": "^1.0.1",
     "postcss": "^5.0.2",