postcss-minify-params 5.1.0 → 5.1.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "postcss-minify-params",
-  "version": "5.1.0",
+  "version": "5.1.3",
   "description": "Minify at-rule params with PostCSS",
   "keywords": [
     "postcss",
@@ -37,6 +37,5 @@
   },
   "peerDependencies": {
     "postcss": "^8.2.15"
-  },
-  "readme": "# postcss-minify-params [![Build Status][ci-img]][ci]\n\n> Minify at-rule params with PostCSS.\n\n```css\n@media only screen   and ( min-width: 400px, min-height: 500px ) {\n    h2{\n        color:blue\n    }\n}\n```\n\n```css\n@media only screen and (min-width:400px,min-height:500px) {\n    h2{\n        color:blue\n    }\n}\n```\n\n## Usage\n\n```js\npostcss([ require('postcss-minify-params') ])\n```\n\nSee [PostCSS] docs for examples for your environment.\n\n## Contributors\n\nSee [CONTRIBUTORS.md](https://github.com/cssnano/cssnano/blob/master/CONTRIBUTORS.md).\n\n## License\n\nMIT © [Bogdan Chadkin](mailto:trysound@yandex.ru)\n\n[PostCSS]: https://github.com/postcss/postcss\n[ci-img]:  https://travis-ci.org/cssnano/postcss-minify-params.svg\n[ci]:      https://travis-ci.org/cssnano/postcss-minify-params\n"
+  }
 }

package/src/index.js

@@ -69,11 +69,21 @@ function transform(legacy, rule) {
   const params = valueParser(rule.params);
 
   params.walk((node, index) => {
-    if (node.type === 'div' || node.type === 'function') {
+    if (node.type === 'div') {
       node.before = node.after = '';
-
+    } else if (node.type === 'function') {
+      node.before = '';
+      if (
+        node.nodes[0] &&
+        node.nodes[0].type === 'word' &&
+        node.nodes[0].value.startsWith('--') &&
+        node.nodes[2] === undefined
+      ) {
+        node.after = ' ';
+      } else {
+        node.after = '';
+      }
       if (
-        node.type === 'function' &&
         node.nodes[4] &&
         node.nodes[0].value.toLowerCase().indexOf('-aspect-ratio') === 3
       ) {
@@ -120,13 +130,7 @@ function transform(legacy, rule) {
   }
 }
 
-/**
- * @param {string} browser
- * @return {boolean}
- */
-function hasAllBug(browser) {
-  return ['ie 10', 'ie 11'].includes(browser);
-}
+const allBugBrowers = new Set(['ie 10', 'ie 11']);
 
 /**
  * @type {import('postcss').PluginCreator<browserslist.Options>}
@@ -140,11 +144,12 @@ function pluginCreator(options = {}) {
     env: options.env,
   });
 
+  const hasAllBug = browsers.some((browser) => allBugBrowers.has(browser));
   return {
     postcssPlugin: 'postcss-minify-params',
 
     OnceExit(css) {
-      css.walkAtRules(transform.bind(null, browsers.some(hasAllBug)));
+      css.walkAtRules((rule) => transform(hasAllBug, rule));
     },
   };
 }