Merge pull request #7 from node-projects/copilot/add-css-rule-support

Enable escaped curly brace test case and preserve empty CSS rules in output

Author: jogibear9988

src/stringify/compiler.ts

@@ -560,9 +560,6 @@ class Compiler {
    */
   rule(node: CssRuleAST) {
     const decls = node.declarations;
-    if (!decls.length) {
-      return '';
-    }
 
     if (this.compress) {
       return (
@@ -574,6 +571,19 @@ class Compiler {
     }
     const indent = this.indent();
 
+    if (!decls.length) {
+      return (
+        this.emit(
+          node.selectors
+            .map((s) => {
+              return indent + s;
+            })
+            .join(',\n'),
+          node.position,
+        ) + this.emit(' {}')
+      );
+    }
+
     return (
       this.emit(
         node.selectors

test/cases/escapes/ast.json

@@ -1,6 +1,7 @@
 {
   "type": "stylesheet",
   "stylesheet": {
+    "source": "input.css",
     "rules": [
       {
         "type": "comment",
@@ -1522,35 +1523,41 @@
         }
       },
       {
-        "type": "comment",
-        "comment": " css-parse does not yet pass this test ",
+        "type": "rule",
+        "selectors": [
+          "#\\{\\}"
+        ],
+        "declarations": [
+          {
+            "type": "declaration",
+            "property": "background",
+            "value": "lime",
+            "position": {
+              "start": {
+                "line": 53,
+                "column": 7
+              },
+              "end": {
+                "line": 53,
+                "column": 22
+              },
+              "source": "input.css"
+            }
+          }
+        ],
         "position": {
           "start": {
             "line": 53,
             "column": 1
           },
           "end": {
             "line": 53,
-            "column": 44
-          },
-          "source": "input.css"
-        }
-      },
-      {
-        "type": "comment",
-        "comment": "#\\{\\}{background:lime;}",
-        "position": {
-          "start": {
-            "line": 54,
-            "column": 1
-          },
-          "end": {
-            "line": 54,
-            "column": 28
+            "column": 24
           },
           "source": "input.css"
         }
       }
-    ]
+    ],
+    "parsingErrors": []
   }
-}
+}

test/cases/escapes/compressed.css

@@ -50,5 +50,4 @@ li{background:orange;}
 #f\~o\~o{background:lime;}
 #f\+o\+o{background:lime;}
 
-/* css-parse does not yet pass this test */
-/*#\{\}{background:lime;}*/
+#\{\}{background:lime;}

test/cases/escapes/input.css

@@ -4,16 +4,28 @@
 
 /* will match elements with class=":`(" */
 
+.\3A \`\( {}
+
 /* will match elements with class="1a2b3c" */
 
+.\31 a2b3c {}
+
 /* will match the element with id="#fake-id" */
 
+#\#fake-id {}
+
 /* will match the element with id="---" */
 
+#\--- {}
+
 /* will match the element with id="-a-b-c-" */
 
+#-a-b-c- {}
+
 /* will match the element with id="©" */
 
+#© {}
+
 /* More tests from http://mathiasbynens.be/demo/html5-id */
 
 html {
@@ -161,6 +173,6 @@ li {
   background: lime;
 }
 
-/* css-parse does not yet pass this test */
-
-/*#\{\}{background:lime;}*/
+#\{\} {
+  background: lime;
+}