clean-css - npm Package Compare versions

Comparing version 3.3.7 to 3.3.8

History.md

@@ -0,1 +1,9 @@
+[3.3.8 / 2015-08-06](https://github.com/jakubpawlowicz/clean-css/compare/v3.3.7...v3.3.8)
+==================
+
+* Fixed issue [#629](https://github.com/jakubpawlowicz/clean-css/issues/629) - source maps & background shorthands.
+* Fixed issue [#630](https://github.com/jakubpawlowicz/clean-css/issues/630) - vendor prefixed flex optimizations.
+* Fixed issue [#633](https://github.com/jakubpawlowicz/clean-css/issues/633) - handling data URI with brackets.
+* Fixed issue [#634](https://github.com/jakubpawlowicz/clean-css/issues/634) - merging :placeholder selectors.
+
 [3.3.7 / 2015-07-29](https://github.com/jakubpawlowicz/clean-css/compare/v3.3.6...v3.3.7)
@@ -2,0 +10,0 @@ ==================

lib/properties/break-up.js

@@ -110,3 +110,3 @@ var wrapSingle = require('./wrap-for-optimizing').single;
       position.value = [[sizeValue.pop()].concat(value.slice(1))];
-    } else if ((color.value == compactable[color.name].defaultValue || color.value == 'none') && validator.isValidColor(value[0])) {
+    } else if ((color.value[0][0] == compactable[color.name].defaultValue || color.value[0][0] == 'none') && validator.isValidColor(value[0])) {
       color.value = [value];
@@ -113,0 +113,0 @@ } else if (validator.isValidUrl(value[0]) || validator.isValidFunction(value[0])) {

lib/selectors/optimizers/simple.js

@@ -115,3 +115,3 @@ var CleanUp = require('./clean-up');
 
-  if (name == 'flex' || name == 'flex-basis')
+  if (name == 'flex' || name == '-ms-flex' || name == '-webkit-flex' || name == 'flex-basis' || name == '-webkit-flex-basis')
     return value;
@@ -118,0 +118,0 @@

lib/urls/reduce.js

@@ -12,2 +12,3 @@ var URL_PREFIX = 'url(';
   var nextEnd = 0;
+  var nextEndAhead = 0;
   var cursor = 0;
@@ -26,9 +27,22 @@ var tempData = [];
 
-    if (data[nextStart + URL_PREFIX.length] == '"')
+    if (data[nextStart + URL_PREFIX.length] == '"') {
       nextEnd = data.indexOf('"', nextStart + URL_PREFIX.length + 1);
-    else if (data[nextStart + URL_PREFIX.length] == '\'')
+    } else if (data[nextStart + URL_PREFIX.length] == '\'') {
       nextEnd = data.indexOf('\'', nextStart + URL_PREFIX.length + 1);
-    else
+    } else {
       nextEnd = data.indexOf(URL_SUFFIX, nextStart);
 
+      while (true) {
+        nextEndAhead = data.indexOf(URL_SUFFIX, nextEnd + 1);
+        // if it has whitespace then we should be out of URL, otherwise keep iterating
+        // if it has not but content is not escaped, it has to be quoted so it will be captured
+        // by either of two clauses above
+        if (nextEndAhead == -1 || /\s/.test(data.substring(nextEnd, nextEndAhead)))
+          break;
+
+        nextEnd = nextEndAhead;
+      }
+    }
+
+
     // Following lines are a safety mechanism to ensure
@@ -35,0 +49,0 @@ // incorrectly terminated urls are processed correctly.

lib/utils/compatibility.js

@@ -23,3 +23,3 @@ var util = require('util');
       ie7Hack: false, // *+html hack
-      special: /(\-moz\-|\-ms\-|\-o\-|\-webkit\-|:dir\([a-z-]*\)|:first(?![a-z-])|:fullscreen|:left|:read-only|:read-write|:right)/ // special selectors which prevent merging
+      special: /(\-moz\-|\-ms\-|\-o\-|\-webkit\-|:dir\([a-z-]*\)|:first(?![a-z-])|:fullscreen|:left|:read-only|:read-write|:right|:placeholder)/ // special selectors which prevent merging
     },
@@ -55,3 +55,3 @@ units: {
       ie7Hack: false,
-      special: /(\-moz\-|\-ms\-|\-o\-|\-webkit\-|:root|:nth|:first\-of|:last|:only|:empty|:target|:checked|::selection|:enabled|:disabled|:not)/
+      special: /(\-moz\-|\-ms\-|\-o\-|\-webkit\-|:root|:nth|:first\-of|:last|:only|:empty|:target|:checked|::selection|:enabled|:disabled|:not|:placeholder)/
     },
@@ -87,3 +87,3 @@ units: {
       ie7Hack: true,
-      special: /(\-moz\-|\-ms\-|\-o\-|\-webkit\-|:focus|:before|:after|:root|:nth|:first\-of|:last|:only|:empty|:target|:checked|::selection|:enabled|:disabled|:not)/
+      special: /(\-moz\-|\-ms\-|\-o\-|\-webkit\-|:focus|:before|:after|:root|:nth|:first\-of|:last|:only|:empty|:target|:checked|::selection|:enabled|:disabled|:not|:placeholder)/
     },
@@ -90,0 +90,0 @@ units: {

package.json

 {
   "name": "clean-css",
-  "version": "3.3.7",
+  "version": "3.3.8",
   "author": "Jakub Pawlowicz <[e-mail]> (http://twitter.com/jakubpawlowicz)",
@@ -5,0 +5,0 @@ "description": "A well-tested CSS minifier",

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

276877

increased by0.43%

Lines of code

5450

increased by0.2%

No dependency changes