Fix prefixing duplicate properties

Fixes #537, closes #553 Co-authored-by: LeoniePhiline <22329650+LeoniePhiline@users.noreply.github.com>
parcel-bundler · Aug 19, 2023 · 94f254c · 94f254c
1 parent 4417942
commit 94f254c
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 4 deletions.
diff --git a/src/lib.rs b/src/lib.rs
@@ -24019,6 +24019,24 @@ mod tests {
         ..Browsers::default()
       },
     );
+    prefix_test(
+      r#"
+      .foo {
+        -webkit-backdrop-filter: blur(8px);
+        backdrop-filter: blur(8px);
+      }
+      "#,
+      indoc! {r#"
+      .foo {
+        -webkit-backdrop-filter: blur(8px);
+        backdrop-filter: blur(8px);
+      }
+      "#},
+      Browsers {
+        safari: Some(16 << 16),
+        ..Browsers::default()
+      },
+    );
 
     prefix_test(
       ".foo { filter: var(--foo) }",

diff --git a/src/properties/prefix_handler.rs b/src/properties/prefix_handler.rs
@@ -95,11 +95,10 @@ macro_rules! define_fallbacks {
           $(
             Property::$name(val $(, mut $p)?) => {
               let mut val = val.clone();
+              $(
+                $p = context.targets.prefixes($p, Feature::$name);
+              )?
               if paste::paste! { self.[<$name:snake>] }.is_none() {
-                $(
-                  $p = context.targets.prefixes($p, Feature::$name);
-                )?
-
                 let fallbacks = val.get_fallbacks(context.targets);
                 #[allow(unused_variables)]
                 let has_fallbacks = !fallbacks.is_empty();