Fix issue where existing non-class declarations are removed

winston0410 · Aug 21, 2021 · ae7e85c · ae7e85c
1 parent 3746677
commit ae7e85c
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/src/tokenizer.js b/src/tokenizer.js
@@ -43,8 +43,6 @@ const hydrateClassCache = (
       tempCache[className],
       tokenizeRules(rule, declarationCache, next, mediaQueryName)
     );
-  } else {
-    //  TODO: handle other selector
   }
 };
 

diff --git a/src/transformer.js b/src/transformer.js
@@ -1,6 +1,6 @@
 import MagicString from "magic-string";
 import { walk } from "svelte/compiler";
-import { assembleRules } from "./helper.js";
+import { assembleRules, getClassName } from "./helper.js";
 
 export default function (code, { dir, base }) {
   const changeable = new MagicString(code);
@@ -17,7 +17,9 @@ export default function (code, { dir, base }) {
             case "Style": {
               for (const child of node.children) {
                 if (child.type === "Rule") {
-                  changeable.overwrite(child.start, child.end, "");
+                  if (getClassName(child)[0]) {
+                    changeable.overwrite(child.start, child.end, "");
+                  }
                 }
               }
               if (node.children.length > 0) {

diff --git a/test/tokenizer.spec.js b/test/tokenizer.spec.js
@@ -254,6 +254,4 @@ describe("when given a rule that uses id as selector", function () {
       })
     );
   });
-
-  it("should be stored in replicate cache", async () => {});
 });
-Original file line number
+Diff line change
@@ Expand Up @@
           })
         );
       });
-      it("should be stored in replicate cache", async () => {});
     });