postcss-merge-longhand - npm Package Compare versions

Comparing version 1.0.1 to 1.0.2

CHANGELOG.md

@@ -0,1 +1,5 @@
+# 1.0.2
+
+* Fixes an issue where properties with the `inherit` keyword were being merged.
+
 # 1.0.1
@@ -2,0 +6,0 @@

dist/lib/canMerge.js

 'use strict';
 
 Object.defineProperty(exports, '__esModule', {
-  value: true
+    value: true
 });
 var important = function important(node) {
-  return node.important;
+    return node.important;
 };
 var unimportant = function unimportant(node) {
-  return !node.important;
+    return !node.important;
 };
+var hasInherit = function hasInherit(node) {
+    return ~node.value.indexOf('inherit');
+};
 
 exports['default'] = function () {
-  for (var _len = arguments.length, props = Array(_len), _key = 0; _key < _len; _key++) {
-    props[_key] = arguments[_key];
-  }
+    for (var _len = arguments.length, props = Array(_len), _key = 0; _key < _len; _key++) {
+        props[_key] = arguments[_key];
+    }
 
-  return props.every(important) || props.every(unimportant);
+    if (props.some(hasInherit)) {
+        return false;
+    }
+    return props.every(important) || props.every(unimportant);
 };
 
 module.exports = exports['default'];

package.json

 {
   "name": "postcss-merge-longhand",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "Merge longhand properties into shorthand with PostCSS.",
@@ -5,0 +5,0 @@ "main": "dist/index.js",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

10809

increased by2.45%

Lines of code

185

increased by3.35%

No dependency changes