autoprefixer 7.1.5 → 7.1.6

package/CHANGELOG.md

@@ -1,6 +1,11 @@
 # Change Log
 This project adheres to [Semantic Versioning](http://semver.org/).
 
+## 7.1.6
+* Add warning for using `browserslist` option instead of `browsers`.
+* Add warning for multiple control comments in the same scope.
+* Fix `Invalid array length` error during indent changes.
+
 ## 7.1.5
 * Fix `::placeholder` prefix for Edge.
 * Fix `inherit`/`initial`/`unset` values for `flex-direction`.

package/lib/autoprefixer.js

@@ -52,6 +52,8 @@ module.exports = postcss.plugin('autoprefixer', function () {
 
     if (options.browser) {
         throw new Error('Change `browser` option to `browsers` in Autoprefixer');
+    } else if (options.browserslist) {
+        throw new Error('Change `browserslist` option to `browsers` in Autoprefixer');
     }
 
     if (options.browsers) {
@@ -81,7 +83,7 @@ module.exports = postcss.plugin('autoprefixer', function () {
         });
         timeCapsule(result, prefixes);
         if (options.remove !== false) {
-            prefixes.processor.remove(css);
+            prefixes.processor.remove(css, result);
         }
         if (options.add !== false) {
             prefixes.processor.add(css, result);

package/lib/declaration.js

@@ -143,7 +143,9 @@ var Declaration = function (_Prefixer) {
         var diff = max - utils.removeNote(prefix).length;
 
         var before = decl.raw('before');
-        before += Array(diff).fill(' ').join('');
+        if (diff > 0) {
+            before += Array(diff).fill(' ').join('');
+        }
 
         return before;
     };

package/lib/processor.js

@@ -32,19 +32,19 @@ var Processor = function () {
 
         css.walkAtRules(function (rule) {
             if (rule.name === 'keyframes') {
-                if (!_this.disabled(rule)) {
+                if (!_this.disabled(rule, result)) {
                     return keyframes && keyframes.process(rule);
                 }
             } else if (rule.name === 'viewport') {
-                if (!_this.disabled(rule)) {
+                if (!_this.disabled(rule, result)) {
                     return viewport && viewport.process(rule);
                 }
             } else if (rule.name === 'supports') {
-                if (_this.prefixes.options.supports !== false && !_this.disabled(rule)) {
+                if (_this.prefixes.options.supports !== false && !_this.disabled(rule, result)) {
                     return supports.process(rule);
                 }
             } else if (rule.name === 'media' && rule.params.indexOf('-resolution') !== -1) {
-                if (!_this.disabled(rule)) {
+                if (!_this.disabled(rule, result)) {
                     return resolution && resolution.process(rule);
                 }
             }
@@ -54,7 +54,7 @@ var Processor = function () {
 
         // Selectors
         css.walkRules(function (rule) {
-            if (_this.disabled(rule)) return undefined;
+            if (_this.disabled(rule, result)) return undefined;
 
             return _this.prefixes.add.selectors.map(function (selector) {
                 return selector.process(rule, result);
@@ -62,7 +62,7 @@ var Processor = function () {
         });
 
         css.walkDecls(function (decl) {
-            if (_this.disabledDecl(decl)) return undefined;
+            if (_this.disabledDecl(decl, result)) return undefined;
 
             if (decl.prop === 'display' && decl.value === 'box') {
                 result.warn('You should write display: flex by final spec ' + 'instead of display: box', { node: decl });
@@ -145,7 +145,7 @@ var Processor = function () {
 
         // Values
         return css.walkDecls(function (decl) {
-            if (_this.disabledValue(decl)) return;
+            if (_this.disabledValue(decl, result)) return;
 
             var unprefixed = _this.prefixes.unprefixed(decl.prop);
             for (var _iterator = _this.prefixes.values('add', unprefixed), _isArray = Array.isArray(_iterator), _i = 0, _iterator = _isArray ? _iterator : _iterator[Symbol.iterator]();;) {
@@ -173,7 +173,7 @@ var Processor = function () {
      */
 
 
-    Processor.prototype.remove = function remove(css) {
+    Processor.prototype.remove = function remove(css, result) {
         var _this2 = this;
 
         // At-rules
@@ -181,7 +181,7 @@ var Processor = function () {
 
         css.walkAtRules(function (rule, i) {
             if (_this2.prefixes.remove['@' + rule.name]) {
-                if (!_this2.disabled(rule)) {
+                if (!_this2.disabled(rule, result)) {
                     rule.parent.removeChild(i);
                 }
             } else if (rule.name === 'media' && rule.params.indexOf('-resolution') !== -1 && resolution) {
@@ -194,7 +194,7 @@ var Processor = function () {
         var _loop = function _loop(checker) {
             css.walkRules(function (rule, i) {
                 if (checker.check(rule)) {
-                    if (!_this2.disabled(rule)) {
+                    if (!_this2.disabled(rule, result)) {
                         rule.parent.removeChild(i);
                     }
                 }
@@ -219,7 +219,7 @@ var Processor = function () {
         }
 
         return css.walkDecls(function (decl, i) {
-            if (_this2.disabled(decl)) return;
+            if (_this2.disabled(decl, result)) return;
 
             var rule = decl.parent;
             var unprefixed = _this2.prefixes.unprefixed(decl.prop);
@@ -299,7 +299,7 @@ var Processor = function () {
      */
 
 
-    Processor.prototype.disabledValue = function disabledValue(node) {
+    Processor.prototype.disabledValue = function disabledValue(node, result) {
         if (this.prefixes.options.grid === false && node.type === 'decl') {
             if (node.prop === 'display' && node.value.indexOf('grid') !== -1) {
                 return true;
@@ -311,7 +311,7 @@ var Processor = function () {
             }
         }
 
-        return this.disabled(node);
+        return this.disabled(node, result);
     };
 
     /**
@@ -319,7 +319,7 @@ var Processor = function () {
      */
 
 
-    Processor.prototype.disabledDecl = function disabledDecl(node) {
+    Processor.prototype.disabledDecl = function disabledDecl(node, result) {
         if (this.prefixes.options.grid === false && node.type === 'decl') {
             if (node.prop.indexOf('grid') !== -1 || node.prop === 'justify-items') {
                 return true;
@@ -332,7 +332,7 @@ var Processor = function () {
             }
         }
 
-        return this.disabled(node);
+        return this.disabled(node, result);
     };
 
     /**
@@ -340,7 +340,7 @@ var Processor = function () {
      */
 
 
-    Processor.prototype.disabled = function disabled(node) {
+    Processor.prototype.disabled = function disabled(node, result) {
         if (node._autoprefixerDisabled !== undefined) {
             return node._autoprefixerDisabled;
         }
@@ -351,29 +351,29 @@ var Processor = function () {
                 if (i.type !== 'comment') {
                     return undefined;
                 }
-                if (/(!\s*)?autoprefixer:\s*off/i.test(i.text)) {
-                    status = false;
-                    return false;
-                } else if (/(!\s*)?autoprefixer:\s*on/i.test(i.text)) {
-                    status = true;
-                    return false;
+                if (/(!\s*)?autoprefixer:\s*(off|on)/i.test(i.text)) {
+                    if (typeof status !== 'undefined') {
+                        result.warn('Second Autoprefixer control comment ' + 'was ignored. Autoprefixer applies control ' + 'comment to whole block, not to next rules.', { node: i });
+                    } else {
+                        status = /on/i.test(i.text);
+                    }
                 }
                 return undefined;
             });
 
-            var result = false;
+            var value = false;
             if (status !== undefined) {
-                result = !status;
+                value = !status;
             } else if (node.parent) {
-                result = this.disabled(node.parent);
+                value = this.disabled(node.parent, result);
             }
 
-            node._autoprefixerDisabled = result;
+            node._autoprefixerDisabled = value;
             return node._autoprefixerDisabled;
         }
 
         if (node.parent) {
-            node._autoprefixerDisabled = this.disabled(node.parent);
+            node._autoprefixerDisabled = this.disabled(node.parent, result);
             return node._autoprefixerDisabled;
         }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "autoprefixer",
-  "version": "7.1.5",
+  "version": "7.1.6",
   "description": "Parse CSS and add vendor prefixes to CSS rules using values from the Can I Use website",
   "keywords": [
     "autoprefixer",
@@ -13,8 +13,8 @@
   "license": "MIT",
   "repository": "postcss/autoprefixer",
   "dependencies": {
-    "browserslist": "^2.5.0",
-    "caniuse-lite": "^1.0.30000744",
+    "browserslist": "^2.5.1",
+    "caniuse-lite": "^1.0.30000748",
     "normalize-range": "^0.1.2",
     "num2fraction": "^1.2.2",
     "postcss": "^6.0.13",
@@ -23,10 +23,10 @@
   "devDependencies": {
     "babel-eslint": "^8.0.1",
     "babel-plugin-transform-class-properties": "^6.24.1",
-    "babel-preset-env": "^1.6.0",
+    "babel-preset-env": "^1.6.1",
     "babelify": "^7.3.0",
-    "browserify": "^14.4.0",
-    "eslint": "^4.8.0",
+    "browserify": "^14.5.0",
+    "eslint": "^4.9.0",
     "eslint-config-postcss": "^2.0.2",
     "fs-extra": "^4.0.2",
     "gulp": "^3.9.1",
@@ -35,7 +35,7 @@
     "gulp-json-editor": "^2.2.1",
     "gulp-replace": "^0.6.1",
     "jest": "^21.2.1",
-    "lint-staged": "^4.2.3",
+    "lint-staged": "^4.3.0",
     "pre-commit": "^1.2.2",
     "size-limit": "^0.11.6",
     "vinyl-source-stream": "^1.1.0",