Fix bug in rewriting of clip paths with transformed path (#988)

Author: Its-Just-Nans

crates/usvg/src/writer.rs

@@ -831,16 +831,22 @@ fn write_group_element(g: &Group, is_clip_path: bool, opt: &WriteOptions, xml: &
         // Same with text. Text elements will be converted into groups,
         // but only the group's children should be written.
         for child in &g.children {
-            if let Node::Path(ref path) = child {
-                let clip_id = g.clip_path.as_ref().map(|cp| cp.id().to_string());
-                write_path(
-                    path,
-                    is_clip_path,
-                    g.transform,
-                    clip_id.as_deref(),
-                    opt,
-                    xml,
-                );
+            match child {
+                Node::Group(child_group) => {
+                    write_group_element(child_group, is_clip_path, opt, xml);
+                }
+                Node::Path(child_path) => {
+                    let clip_id = g.clip_path.as_ref().map(|cp| cp.id().to_string());
+                    write_path(
+                        child_path,
+                        is_clip_path,
+                        g.transform,
+                        clip_id.as_deref(),
+                        opt,
+                        xml,
+                    );
+                }
+                _ => {}
             }
         }
         return;

crates/usvg/tests/files/clip-path-with-transform-expected.svg

@@ -77,6 +77,11 @@ fn text_simple_case() {
     resave("text-simple-case");
 }
 
+#[test]
+fn clip_path_with_transform() {
+    resave("clip-path-with-transform");
+}
+
 #[test]
 fn preserve_id_filter() {
     resave("preserve-id-filter");