Handle CSS-comments better in the preprocess-function (PR 14963 follo…

…w-up) This fixes another oversight, please see the updated tests.
mozilla · Jun 2, 2022 · 397f2e6 · 397f2e6
1 parent 1ac33c9
commit 397f2e6
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/external/builder/builder.js b/external/builder/builder.js
@@ -209,7 +209,10 @@ function preprocess(inFilename, outFilename, defines) {
  !stack.includes(STATE_ELSE_FALSE)
  ) {
  writeLine(
- line.replace(/^\/\/|^\/\*|^<!--/g, " ").replace(/\*\/$|-->$/g, "")
+ line
+ .replace(/^\/\/|^<!--/g, " ")
+ .replace(/(^\s*)\/\*/g, "$1 ")
+ .replace(/\*\/$|-->$/g, "")
  );
  }
  }

diff --git a/external/builder/fixtures/css-comment-expected.css b/external/builder/fixtures/css-comment-expected.css
@@ -1,4 +1,5 @@
 /* Comment here... */
  div {
  margin: 0;
+ padding: 0;
  }
diff --git a/external/builder/fixtures/css-comment.css b/external/builder/fixtures/css-comment.css
@@ -2,5 +2,6 @@
 /*#if TRUE*/
 /*div {*/
 /* margin: 0;*/
+ /*padding: 0;*/
 /*}*/
 /*#endif*/