Keep hidden elements if any descendant enables visibility

TrySound · TrySound · commit d06747abca1e · 2021-02-25T00:05:26.000+03:00
diff --git a/plugins/removeHiddenElems.js b/plugins/removeHiddenElems.js
@@ -52,7 +52,9 @@ exports.fn = function (item, params) {
         // https://www.w3schools.com/cssref/pr_class_visibility.asp
         if (
             params.isHidden &&
-            item.hasAttr('visibility', 'hidden')
+            item.hasAttr('visibility', 'hidden') &&
+            // keep if any descendant enables visibility
+            item.querySelector('[visibility=visible]') == null
         ) return false;
 
         // display="none"
diff --git a/test/plugins/removeHiddenElems.12.svg b/test/plugins/removeHiddenElems.12.svg
@@ -0,0 +1,20 @@
+<svg width="480" height="360" xmlns="http://www.w3.org/2000/svg">
+  <rect x="96" y="96" width="96" height="96" fill="lime" />
+  <g visibility="hidden">
+    <rect x="96" y="96" width="96" height="96" fill="red" />
+  </g>
+  <rect x="196.5" y="196.5" width="95" height="95" fill="red"/>
+  <g visibility="hidden">
+    <rect x="196" y="196" width="96" height="96" fill="lime" visibility="visible" />
+  </g>
+</svg>
+
+@@@
+
+<svg width="480" height="360" xmlns="http://www.w3.org/2000/svg">
+    <rect x="96" y="96" width="96" height="96" fill="lime"/>
+    <rect x="196.5" y="196.5" width="95" height="95" fill="red"/>
+    <g visibility="hidden">
+        <rect x="196" y="196" width="96" height="96" fill="lime" visibility="visible"/>
+    </g>
+</svg>
