From: GoalSmashers <jakub@goalsmashers.com>
Date: Sat, 16 Nov 2013 09:59:18 +0000 (+0100)
Subject: Fixes #165 - adds removing extra space after rgba/hsla declarations.
X-Git-Url: https://git.ndcode.org/public/gitweb.cgi?a=commitdiff_plain;h=e3a0be7c10c4282ac38aa3ea883bed5beaf18e10;p=clean-css.git

Fixes #165 - adds removing extra space after rgba/hsla declarations.
---

diff --git a/History.md b/History.md
index 1297c532..78a675d5 100644
--- a/History.md
+++ b/History.md
@@ -1,3 +1,8 @@
+[2.1.0 / 2013-xx-xx (UNRELEASED)](https://github.com/GoalSmashers/clean-css/compare/v2.0.0...HEAD)
+==================
+
+* Fixed issue [#165](https://github.com/GoalSmashers/clean-css/issues/165) - extra space after trailing parenthesis.
+
 [2.0.1 / 2013-11-14](https://github.com/GoalSmashers/clean-css/compare/v2.0.0...v2.0.1)
 ==================
 
diff --git a/lib/clean.js b/lib/clean.js
index 9b1f4ffc..5238595c 100644
--- a/lib/clean.js
+++ b/lib/clean.js
@@ -255,6 +255,9 @@ module.exports = function(options) {
       return match.replace(/\+/g, ' + ');
     });
 
+    // remove space after (rgba|hsla) declaration - see #165
+    replace(/(rgba|hsla)\(([^\)]+)\) /g, '$1($2)');
+
     if (!options.noAdvanced) {
       replace(function optimizeSelectors() {
         data = new SelectorsOptimizer(data, {
diff --git a/test/unit-test.js b/test/unit-test.js
index 42eba554..d0b95a4d 100644
--- a/test/unit-test.js
+++ b/test/unit-test.js
@@ -176,6 +176,22 @@ vows.describe('clean-units').addBatch({
     'in content preceded by line break': [
       '.content{display:block}#foo{' + lineBreak + 'content:"x"}',
       '.content{display:block}#foo{content:"x"}'
+    ],
+    'after rgb': [
+      'a{text-shadow:rgb(255,0,1) 1px 1px}',
+      'a{text-shadow:#ff0001 1px 1px}'
+    ],
+    'after rgba': [
+      'a{text-shadow:rgba(255,0,0,1) 0 1px}',
+      'a{text-shadow:rgba(255,0,0,1)0 1px}'
+    ],
+    'after hsl': [
+      'a{text-shadow:hsl(240,100%,40%) -1px 1px}',
+      'a{text-shadow:#00c -1px 1px}'
+    ],
+    'after hsla': [
+      'a{text-shadow:hsla(240,100%,40%,.5) -1px 1px}',
+      'a{text-shadow:hsla(240,100%,40%,.5)-1px 1px}'
     ]
   }),
   'line breaks': cssContext({