properly ignore embedded imports in data urls

postcss · Aug 9, 2023 · 3449081 · 3449081
1 parent a78c387
commit 3449081
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/lib/parse-styles.js b/lib/parse-styles.js
@@ -92,9 +92,11 @@ async function resolveImportId(result, stmt, options, state, postcss) {
 
     return
   } else if (dataURL.isValid(stmt.from)) {
-    // Data urls can't be used a base url to resolve imports.
+    // Data urls can't be used as a base url to resolve imports.
     // When the parent statement has a data url
-    // and the current statement doesn't have a data url we ignore the statement.
+    // and the current statement doesn't have a data url
+    // we set the conditions to `not all`.
+    stmt.media = ["not all"]
     return
   }
 

diff --git a/test/fixtures/data-url.expected.css b/test/fixtures/data-url.expected.css
@@ -1,4 +1,4 @@
-@import url(foo.css);p { color: green; }p { color: blue; }@media (min-width: 320px){@layer foo{
+@import url(foo.css) not all;p { color: green; }p { color: blue; }@media (min-width: 320px){@layer foo{
 p { color: green; } } }@media (min-width: 320px){@layer foo{
 
 p { color: blue; } } }/* Mixed imports: */p {