From: Jakub Pawlowicz <contact@jakubpawlowicz.com>
Date: Wed, 4 Mar 2015 10:10:17 +0000 (+0000)
Subject: Fixes #472 - broken function minification.
X-Git-Url: https://git.ndcode.org/public/gitweb.cgi?a=commitdiff_plain;h=5cbc13672eaacf3e07d674697188a30ef5d939cb;p=clean-css.git

Fixes #472 - broken function minification.

There was a wrong tokenization inside functions (gradient) with
functions (rgb) inside them.
---

diff --git a/History.md b/History.md
index b75a7e38..5b3aa101 100644
--- a/History.md
+++ b/History.md
@@ -6,6 +6,7 @@
 [3.1.4 / 2015-xx-xx](https://github.com/jakubpawlowicz/clean-css/compare/v3.1.3...3.1)
 ==================
 
+* Fixes issue [#472](https://github.com/jakubpawlowicz/clean-css/issues/472) - broken function minification.
 * Fixes issue [#477](https://github.com/jakubpawlowicz/clean-css/issues/477) - `@import`s order in restructuring.
 * Fixes issue [#478](https://github.com/jakubpawlowicz/clean-css/issues/478) - ultimate fix to brace whitespace.
 
diff --git a/lib/utils/extractors.js b/lib/utils/extractors.js
index 64d54f7a..824f179a 100644
--- a/lib/utils/extractors.js
+++ b/lib/utils/extractors.js
@@ -22,7 +22,7 @@ var Extractors = {
     var addSourceMap = context.addSourceMap;
 
     if (string.replace && string.indexOf(')') > 0)
-      string = string.replace(/\)([^\s_;:,])/g, context.addSourceMap ? ') __ESCAPED_COMMENT_CLEAN_CSS(0,-1)__$1' : ') $1');
+      string = string.replace(/\)([^\s_;:,\)])/g, context.addSourceMap ? ') __ESCAPED_COMMENT_CLEAN_CSS(0,-1)__$1' : ') $1');
 
     for (var i = 0, l = string.length; i < l; i++) {
       current = string[i];
diff --git a/test/source-map-test.js b/test/source-map-test.js
index a46d69b4..a3006abe 100644
--- a/test/source-map-test.js
+++ b/test/source-map-test.js
@@ -22,6 +22,12 @@ vows.describe('source-map')
       'gets right output': function (minified) {
         assert.equal(minified.styles, 'html{font-family:sans-serif;-ms-text-size-adjust:100%;-webkit-text-size-adjust:100%}');
       }
+    },
+    'background gradient': {
+      'topic': new CleanCSS({ sourceMap: true }).minify('a{background: linear-gradient(to bottom, rgba(0, 0, 0, 0.1) 0, rgba(0, 0, 0, 0.1))}'),
+      'gets right output': function (minified) {
+        assert.equal(minified.styles, 'a{background:linear-gradient(to bottom,rgba(0,0,0,.1) 0,rgba(0,0,0,.1))}');
+      }
     }
   })
   .addBatch({