stylehacks 2.1.0 → 2.3.2

package/CHANGELOG.md

@@ -1,3 +1,30 @@
+# 2.3.2
+
+* Resolves an issue where stylehacks would crash on CSS mixins.
+
+# 2.3.1
+
+* Upgraded postcss-selector-parser to `v2.0.0`.
+* Fixed an issue where stylehacks was not removing `*zoom: 1` from rules
+  generated by other PostCSS plugins.
+
+# 2.3.0
+
+* Each warning now contains more information about the hack; what identifier
+  it targets, what the hack is, and the browsers that it affects. E.g:
+  
+  ```js
+  {
+      browsers: [ 'ie 6', 'ie 5.5' ],
+      identifier: 'selector',
+      hack: '* html h1'
+  }
+  ```
+
+# 2.2.0
+
+* Added `value\9` hack.
+
 # 2.1.0
 
 * Added `html ~ /**/ body .selector` hack.

package/README.md

@@ -2,6 +2,7 @@
 
 > Detect/remove browser hacks from CSS files.
 
+
 ## Install
 
 With [npm](https://npmjs.org/package/stylehacks) do:
@@ -10,6 +11,7 @@ With [npm](https://npmjs.org/package/stylehacks) do:
 npm install stylehacks --save
 ```
 
+
 ## Example
 
 In its default mode, stylehacks will remove hacks from your CSS file, based on
@@ -32,11 +34,12 @@ h1 {
 }
 ```
 
+
 ## API
 
 ### `stylehacks.detect(node)`
 
-Type: `function`
+Type: `function`  
 Returns: `boolean`
 
 This method will take any PostCSS *node*, run applicable plugins depending on
@@ -55,7 +58,7 @@ h1 { _color: red }
 
 ##### browsers
 
-Type: `string|array`
+Type: `string|array`  
 Default: [browserslist defaults](https://github.com/ai/browserslist)
 
 Specify the browsers that you wish to support. The string will be passed
@@ -64,7 +67,7 @@ stylehacks will use it instead of parsing the browsers itself.
 
 ##### lint
 
-Type: `boolean`
+Type: `boolean`  
 Default: `false`
 
 If lint mode is enabled, stylehacks will not remove hacks from the CSS; instead,
@@ -73,7 +76,7 @@ you are expected to handle these messages yourself.
 
 ##### silent
 
-Type: `boolean`
+Type: `boolean`  
 Default: `false`
 
 Used in combination with the lint option; disables all logging. When using the
@@ -81,7 +84,7 @@ CLI, the process will exit with 0 or 1 as usual.
 
 ##### sourcemap
 
-Type: `boolean`
+Type: `boolean`  
 Default: `false`
 
 Generate a sourcemap with the transformed CSS.
@@ -100,6 +103,7 @@ stylehacks also ships with a CLI app. To see the available options, just run:
 $ stylehacks --help
 ```
 
+
 ## Related
 
 stylehacks works well with your existing PostCSS setup:
@@ -107,15 +111,18 @@ stylehacks works well with your existing PostCSS setup:
 * [stylelint] - Comprehensive & modern CSS linter, to ensure that your code
   style rules are respected.
 
+
 ## Contributing
 
 Pull requests are welcome. If you add functionality, then please add unit tests
 to cover it.
 
+
 ## License
 
 MIT © [Ben Briggs](http://beneb.info)
 
+
 [ci]:        https://travis-ci.org/ben-eb/stylehacks
 [deps]:      https://gemnasium.com/ben-eb/stylehacks
 [npm]:       http://badge.fury.io/js/stylehacks

package/dist/cli.js

@@ -0,0 +1,69 @@
+#!/usr/bin/env node
+'use strict';
+
+var _fs = require('fs');
+
+var _fs2 = _interopRequireDefault(_fs);
+
+var _readFileStdin = require('read-file-stdin');
+
+var _readFileStdin2 = _interopRequireDefault(_readFileStdin);
+
+var _writeFileStdout = require('write-file-stdout');
+
+var _writeFileStdout2 = _interopRequireDefault(_writeFileStdout);
+
+var _minimist = require('minimist');
+
+var _minimist2 = _interopRequireDefault(_minimist);
+
+var _package = require('../package.json');
+
+var _ = require('./');
+
+var _2 = _interopRequireDefault(_);
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+
+var opts = (0, _minimist2.default)(process.argv.slice(2), {
+    alias: {
+        b: 'browsers',
+        l: 'lint',
+        h: 'help',
+        s: 'sourcemap',
+        v: 'version'
+    }
+});
+
+if (opts.version) {
+    console.log(_package.version);
+} else {
+    var file = opts._[0];
+    var out = opts._[1];
+
+    if (file === 'help' || opts.help) {
+        _fs2.default.createReadStream(__dirname + '/../usage.txt').pipe(process.stdout).on('close', function () {
+            return process.exit(1);
+        });
+    } else {
+        (0, _readFileStdin2.default)(file, function (err, buf) {
+            if (err) {
+                throw err;
+            }
+            if (file) {
+                opts.from = file;
+            }
+            if (out) {
+                opts.to = out;
+            }
+            _2.default.process(String(buf), opts).then(function (result) {
+                if (result.warnings().length) {
+                    process.exit(1);
+                }
+                if (!opts.lint) {
+                    (0, _writeFileStdout2.default)(result.css);
+                }
+            });
+        });
+    }
+}

package/dist/dictionary/browsers.js

@@ -0,0 +1,9 @@
+'use strict';
+
+exports.__esModule = true;
+var FF_2 = exports.FF_2 = 'firefox 2';
+var IE_5_5 = exports.IE_5_5 = 'ie 5.5';
+var IE_6 = exports.IE_6 = 'ie 6';
+var IE_7 = exports.IE_7 = 'ie 7';
+var IE_8 = exports.IE_8 = 'ie 8';
+var OP_9 = exports.OP_9 = 'opera 9';

package/dist/dictionary/identifiers.js

@@ -0,0 +1,7 @@
+'use strict';
+
+exports.__esModule = true;
+var MEDIA_QUERY = exports.MEDIA_QUERY = 'media query';
+var PROPERTY = exports.PROPERTY = 'property';
+var SELECTOR = exports.SELECTOR = 'selector';
+var VALUE = exports.VALUE = 'value';

package/dist/dictionary/postcss.js

@@ -0,0 +1,6 @@
+'use strict';
+
+exports.__esModule = true;
+var ATRULE = exports.ATRULE = 'atrule';
+var DECL = exports.DECL = 'decl';
+var RULE = exports.RULE = 'rule';

package/dist/dictionary/tags.js

@@ -0,0 +1,5 @@
+'use strict';
+
+exports.__esModule = true;
+var BODY = exports.BODY = 'body';
+var HTML = exports.HTML = 'html';

package/dist/exists.js

@@ -1,13 +1,9 @@
-'use strict';
-
-Object.defineProperty(exports, '__esModule', {
-    value: true
-});
-exports['default'] = exists;
+"use strict";
 
+exports.__esModule = true;
+exports.default = exists;
 function exists(selector, index, value) {
     var node = selector.at(index);
     return node && node.value === value;
 }
-
-module.exports = exports['default'];
+module.exports = exports["default"];

package/dist/formatter.js

@@ -1,19 +1,15 @@
 'use strict';
 
-Object.defineProperty(exports, '__esModule', {
-    value: true
-});
+exports.__esModule = true;
 
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { 'default': obj }; }
+var _path = require('path');
+
+var _path2 = _interopRequireDefault(_path);
 
 var _chalk = require('chalk');
 
 var _chalk2 = _interopRequireDefault(_chalk);
 
-var _path = require('path');
-
-var _path2 = _interopRequireDefault(_path);
-
 var _textTable = require('text-table');
 
 var _textTable2 = _interopRequireDefault(_textTable);
@@ -26,29 +22,31 @@ var _plur = require('plur');
 
 var _plur2 = _interopRequireDefault(_plur);
 
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
+
 var logFrom = function logFrom(fromValue) {
     if (!fromValue.indexOf('<')) {
         return fromValue;
     }
-    return _path2['default'].relative(process.cwd(), fromValue);
+    return _path2.default.relative(process.cwd(), fromValue);
 };
 
 var hacksFound = function hacksFound(messages) {
-    var num = messages.length + (0, _plur2['default'])(' hack', messages.length);
-    return '\n\n  ' + _logSymbols2['default'].error + '  ' + num + ' found.\n';
+    var num = messages.length + (0, _plur2.default)(' hack', messages.length);
+    return '\n\n  ' + _logSymbols2.default.error + '  ' + num + ' found.\n';
 };
 
-exports['default'] = function (input) {
+exports.default = function (input) {
     var messages = input.messages;
     var source = input.source;
 
     if (!messages.length) {
-        return '  ' + _logSymbols2['default'].success + '  No hacks found.';
+        return '  ' + _logSymbols2.default.success + '  No hacks found.';
     }
 
-    return _chalk2['default'].underline(logFrom(source)) + '\n' + (0, _textTable2['default'])(messages.map(function (msg) {
+    return _chalk2.default.underline(logFrom(source)) + '\n' + (0, _textTable2.default)(messages.map(function (msg) {
         var parts = msg.text.split(': ');
-        return ['', _chalk2['default'].gray('line ' + msg.node.source.start.line), _chalk2['default'].gray('col ' + msg.node.source.start.column), parts[0], _chalk2['default'].red(parts[1])];
+        return ['', _chalk2.default.gray('line ' + msg.node.source.start.line), _chalk2.default.gray('col ' + msg.node.source.start.column), parts[0], _chalk2.default.red(parts[1])];
     })) + hacksFound(messages);
 };
 

package/dist/index.js

@@ -1,10 +1,6 @@
 'use strict';
 
-Object.defineProperty(exports, '__esModule', {
-    value: true
-});
-
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { 'default': obj }; }
+exports.__esModule = true;
 
 var _postcss = require('postcss');
 
@@ -22,71 +18,86 @@ var _formatter = require('./formatter');
 
 var _formatter2 = _interopRequireDefault(_formatter);
 
-// plugins
+var _bodyEmpty = require('./plugins/bodyEmpty');
+
+var _bodyEmpty2 = _interopRequireDefault(_bodyEmpty);
+
+var _htmlCombinatorCommentBody = require('./plugins/htmlCombinatorCommentBody');
 
-var _pluginsBodyEmpty = require('./plugins/bodyEmpty');
+var _htmlCombinatorCommentBody2 = _interopRequireDefault(_htmlCombinatorCommentBody);
 
-var _pluginsBodyEmpty2 = _interopRequireDefault(_pluginsBodyEmpty);
+var _htmlFirstChild = require('./plugins/htmlFirstChild');
 
-var _pluginsHtmlCombinatorCommentBody = require('./plugins/htmlCombinatorCommentBody');
+var _htmlFirstChild2 = _interopRequireDefault(_htmlFirstChild);
 
-var _pluginsHtmlCombinatorCommentBody2 = _interopRequireDefault(_pluginsHtmlCombinatorCommentBody);
+var _important = require('./plugins/important');
 
-var _pluginsHtmlFirstChild = require('./plugins/htmlFirstChild');
+var _important2 = _interopRequireDefault(_important);
 
-var _pluginsHtmlFirstChild2 = _interopRequireDefault(_pluginsHtmlFirstChild);
+var _leadingStar = require('./plugins/leadingStar');
 
-var _pluginsImportant = require('./plugins/important');
+var _leadingStar2 = _interopRequireDefault(_leadingStar);
 
-var _pluginsImportant2 = _interopRequireDefault(_pluginsImportant);
+var _leadingUnderscore = require('./plugins/leadingUnderscore');
 
-var _pluginsLeadingStar = require('./plugins/leadingStar');
+var _leadingUnderscore2 = _interopRequireDefault(_leadingUnderscore);
 
-var _pluginsLeadingStar2 = _interopRequireDefault(_pluginsLeadingStar);
+var _mediaSlash = require('./plugins/mediaSlash0');
 
-var _pluginsLeadingUnderscore = require('./plugins/leadingUnderscore');
+var _mediaSlash2 = _interopRequireDefault(_mediaSlash);
 
-var _pluginsLeadingUnderscore2 = _interopRequireDefault(_pluginsLeadingUnderscore);
+var _mediaSlash3 = require('./plugins/mediaSlash9');
 
-var _pluginsMediaSlash0 = require('./plugins/mediaSlash0');
+var _mediaSlash4 = _interopRequireDefault(_mediaSlash3);
 
-var _pluginsMediaSlash02 = _interopRequireDefault(_pluginsMediaSlash0);
+var _slash = require('./plugins/slash9');
 
-var _pluginsMediaSlash9 = require('./plugins/mediaSlash9');
+var _slash2 = _interopRequireDefault(_slash);
 
-var _pluginsMediaSlash92 = _interopRequireDefault(_pluginsMediaSlash9);
+var _starHtml = require('./plugins/starHtml');
 
-var _pluginsStarHtml = require('./plugins/starHtml');
+var _starHtml2 = _interopRequireDefault(_starHtml);
 
-var _pluginsStarHtml2 = _interopRequireDefault(_pluginsStarHtml);
+var _trailingSlashComma = require('./plugins/trailingSlashComma');
 
-var _pluginsTrailingSlashComma = require('./plugins/trailingSlashComma');
+var _trailingSlashComma2 = _interopRequireDefault(_trailingSlashComma);
 
-var _pluginsTrailingSlashComma2 = _interopRequireDefault(_pluginsTrailingSlashComma);
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
-var plugins = [_pluginsBodyEmpty2['default'], _pluginsHtmlCombinatorCommentBody2['default'], _pluginsHtmlFirstChild2['default'], _pluginsImportant2['default'], _pluginsLeadingStar2['default'], _pluginsLeadingUnderscore2['default'], _pluginsMediaSlash02['default'], _pluginsMediaSlash92['default'], _pluginsStarHtml2['default'], _pluginsTrailingSlashComma2['default']];
+var plugins = [_bodyEmpty2.default, _htmlCombinatorCommentBody2.default, _htmlFirstChild2.default, _important2.default, _leadingStar2.default, _leadingUnderscore2.default, _mediaSlash2.default, _mediaSlash4.default, _slash2.default, _starHtml2.default, _trailingSlashComma2.default];
 
-var stylehacks = _postcss2['default'].plugin('stylehacks', function () {
-    var opts = arguments.length <= 0 || arguments[0] === undefined ? {} : arguments[0];
+// plugins
+
+
+var stylehacks = _postcss2.default.plugin('stylehacks', function () {
+    var opts = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
 
     var b = opts.browsers;
-    var browsers = b instanceof Array ? b : (0, _browserslist2['default'])(b);
+    var browsers = b instanceof Array ? b : (0, _browserslist2.default)(b);
 
     return function (css, result) {
-        plugins.forEach(function (Plugin) {
-            var hack = new Plugin(css, result);
+        var processors = plugins.reduce(function (list, Plugin) {
+            var hack = new Plugin(result);
             var applied = browsers.some(function (browser) {
                 return hack.targets.some(function (target) {
                     return browser === target;
                 });
             });
             if (applied) {
-                return;
+                return list;
             }
-            if (opts.lint) {
-                return hack.detectAndWarn();
-            }
-            return hack.detectAndResolve();
+            return [].concat(list, [hack]);
+        }, []);
+        css.walk(function (node) {
+            processors.forEach(function (proc) {
+                if (!~proc.nodeTypes.indexOf(node.type)) {
+                    return;
+                }
+                if (opts.lint) {
+                    return proc.detectAndWarn(node);
+                }
+                return proc.detectAndResolve(node);
+            });
         });
     };
 });
@@ -101,17 +112,17 @@ stylehacks.detect = function (node) {
 };
 
 stylehacks.process = function (css) {
-    var opts = arguments.length <= 1 || arguments[1] === undefined ? {} : arguments[1];
+    var opts = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
 
     opts.reporter = {};
-    opts.reporter.formatter = _formatter2['default'];
+    opts.reporter.formatter = _formatter2.default;
     opts.map = opts.map || (opts.sourcemap ? true : null);
-    var processor = (0, _postcss2['default'])([stylehacks(opts)]);
+    var processor = (0, _postcss2.default)([stylehacks(opts)]);
     if (opts.lint && !opts.silent) {
-        processor.use((0, _postcssReporter2['default'])(opts.reporter));
+        processor.use((0, _postcssReporter2.default)(opts.reporter));
     }
     return processor.process(css, opts);
 };
 
-exports['default'] = stylehacks;
+exports.default = stylehacks;
 module.exports = exports['default'];

package/dist/isMixin.js

@@ -0,0 +1,14 @@
+'use strict';
+
+exports.__esModule = true;
+exports.default = isMixin;
+function isMixin(node) {
+    var selector = node.selector;
+    // If the selector ends with a ':' it is likely a part of a custom mixin.
+
+    if (!selector || selector[selector.length - 1] === ':') {
+        return true;
+    }
+    return false;
+}
+module.exports = exports['default'];

package/dist/plugin.js

@@ -1,88 +1,85 @@
-'use strict';
+"use strict";
 
-Object.defineProperty(exports, '__esModule', {
-    value: true
-});
+exports.__esModule = true;
 
-var _createClass = (function () { function defineProperties(target, props) { for (var i = 0; i < props.length; i++) { var descriptor = props[i]; descriptor.enumerable = descriptor.enumerable || false; descriptor.configurable = true; if ('value' in descriptor) descriptor.writable = true; Object.defineProperty(target, descriptor.key, descriptor); } } return function (Constructor, protoProps, staticProps) { if (protoProps) defineProperties(Constructor.prototype, protoProps); if (staticProps) defineProperties(Constructor, staticProps); return Constructor; }; })();
+var _extends = Object.assign || function (target) { for (var i = 1; i < arguments.length; i++) { var source = arguments[i]; for (var key in source) { if (Object.prototype.hasOwnProperty.call(source, key)) { target[key] = source[key]; } } } return target; };
 
-exports['default'] = plugin;
+exports.default = plugin;
 
-function _classCallCheck(instance, Constructor) { if (!(instance instanceof Constructor)) { throw new TypeError('Cannot call a class as a function'); } }
+function _classCallCheck(instance, Constructor) { if (!(instance instanceof Constructor)) { throw new TypeError("Cannot call a class as a function"); } }
 
-function plugin(targets, nodeTypes, _detect) {
-    var Plugin = (function () {
-        function Plugin(css, result) {
+function plugin(targets, nodeTypes, detect) {
+    var Plugin = function () {
+        function Plugin(result) {
             _classCallCheck(this, Plugin);
 
             this.nodes = [];
-            this.css = css;
             this.result = result;
             this.targets = targets;
             this.nodeTypes = nodeTypes;
         }
 
-        _createClass(Plugin, [{
-            key: 'push',
-            value: function push(node, message) {
-                node._stylehacks = message;
-                this.nodes.push(node);
-            }
-        }, {
-            key: 'any',
-            value: function any(node) {
-                var hasHack = false;
-                if (~this.nodeTypes.indexOf(node.type)) {
-                    _detect.apply(this, arguments);
-                    if (node._stylehacks) {
-                        hasHack = true;
-                    }
-                }
-                return hasHack;
-            }
-        }, {
-            key: 'detect',
-            value: function detect() {
-                this.css.walk((function (node) {
-                    if (~this.nodeTypes.indexOf(node.type)) {
-                        _detect.apply(this, arguments);
-                    }
-                }).bind(this));
-            }
-        }, {
-            key: 'detectAndResolve',
-            value: function detectAndResolve() {
-                this.detect();
-                return this.resolve();
-            }
-        }, {
-            key: 'detectAndWarn',
-            value: function detectAndWarn() {
-                this.detect();
-                return this.warn();
+        Plugin.prototype.push = function push(node, metadata) {
+            node._stylehacks = _extends({}, metadata, {
+                message: "Bad " + metadata.identifier + ": " + metadata.hack,
+                browsers: this.targets
+            });
+            this.nodes.push(node);
+        };
+
+        Plugin.prototype.any = function any(node) {
+            if (~this.nodeTypes.indexOf(node.type)) {
+                detect.apply(this, arguments);
+                return !!node._stylehacks;
             }
-        }, {
-            key: 'resolve',
-            value: function resolve() {
-                return this.nodes.forEach(function (node) {
-                    return node.remove();
-                });
+            return false;
+        };
+
+        Plugin.prototype.detectAndResolve = function detectAndResolve() {
+            this.nodes = [];
+
+            for (var _len = arguments.length, args = Array(_len), _key = 0; _key < _len; _key++) {
+                args[_key] = arguments[_key];
             }
-        }, {
-            key: 'warn',
-            value: function warn() {
-                var _this = this;
-
-                return this.nodes.forEach(function (node) {
-                    return _this.result.warn(node._stylehacks, { node: node });
-                });
+
+            detect.apply(this, args);
+            return this.resolve();
+        };
+
+        Plugin.prototype.detectAndWarn = function detectAndWarn() {
+            this.nodes = [];
+
+            for (var _len2 = arguments.length, args = Array(_len2), _key2 = 0; _key2 < _len2; _key2++) {
+                args[_key2] = arguments[_key2];
             }
-        }]);
+
+            detect.apply(this, args);
+            return this.warn();
+        };
+
+        Plugin.prototype.resolve = function resolve() {
+            return this.nodes.forEach(function (node) {
+                return node.remove();
+            });
+        };
+
+        Plugin.prototype.warn = function warn() {
+            var _this = this;
+
+            return this.nodes.forEach(function (node) {
+                var _node$_stylehacks = node._stylehacks,
+                    message = _node$_stylehacks.message,
+                    browsers = _node$_stylehacks.browsers,
+                    identifier = _node$_stylehacks.identifier,
+                    hack = _node$_stylehacks.hack;
+
+                return node.warn(_this.result, message, { browsers: browsers, identifier: identifier, hack: hack });
+            });
+        };
 
         return Plugin;
-    })();
+    }();
 
     return Plugin;
 }
-
-module.exports = exports['default'];
+module.exports = exports["default"];