Don't remove an empty container if parent is "switch"

Fix #1410
svg · Feb 6, 2023 · 52ce412 · 52ce412
1 parent aa5d667
commit 52ce412
Show file tree

Hide file tree

Showing 3 changed files with 31 additions and 0 deletions.
diff --git a/plugins/removeEmptyContainers.js b/plugins/removeEmptyContainers.js
@@ -49,6 +49,10 @@ exports.fn = () => {
         if (node.name === 'mask' && node.attributes.id != null) {
           return;
         }
+        // if parent is switch, don't remove
+        if (parentNode.name === 'switch') {
+          return;
+        }
         detachNodeFromParent(node, parentNode);
       },
     },

diff --git a/test/svgo/_index.test.js b/test/svgo/_index.test.js
@@ -61,4 +61,12 @@ describe('svgo', () => {
     });
     expect(normalize(result.data)).toEqual(expected);
   });
+  it('should keep an empty container inside switch tag', async () => {
+    const [original, expected] = await parseFixture('switch-emptycontainer.svg');
+    const result = optimize(original, {
+      path: 'input.svg',
+      js2svg: { pretty: true },
+    });
+    expect(normalize(result.data)).toEqual(expected);
+  });
 });
diff --git a/test/svgo/switch-emptycontainer.svg b/test/svgo/switch-emptycontainer.svg