#378: Hack for braces in space-between-declarations

csscomb · May 20, 2015 · 3717cba · 3717cba
1 parent 3204363
commit 3717cba
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 0 deletions.
diff --git a/lib/options/space-between-declarations.js b/lib/options/space-between-declarations.js
@@ -66,6 +66,8 @@ module.exports = (function() {
             var value = this.getValue('space-between-declarations');
 
             // TODO: Limit nodes to blocks, stylesheet, etc.
+            // XXX: Hack for braces
+            if (node.is('braces')) return;
 
             for (var i = 0, l = node.length; i < l; i++) {
                 if (!node.get(i) || !node.get(i).is('declarationDelimiter')) continue;

diff --git a/test/options/space-between-declarations/issue-378.css b/test/options/space-between-declarations/issue-378.css
@@ -0,0 +1,3 @@
+@media (min-width: 768px) {
+    body { background: red; }
+}
diff --git a/test/options/space-between-declarations/test.js b/test/options/space-between-declarations/test.js
@@ -38,4 +38,9 @@ describe('options/space-between-declarations:', function() {
         this.comb.configure({ 'space-between-declarations': '\n    ' });
         this.shouldBeEqual('issue-239.css', 'issue-239.expected.css');
     });
+
+    it('Issue 378', function() {
+        this.comb.configure({ 'space-between-declarations': '\n' });
+        this.shouldBeEqual('issue-378.css');
+    });
 });