Prevent removing IDs in SVGs only with defs

svg · Oct 22, 2017 · f8234fd · f8234fd
1 parent 3b6fa1e
commit f8234fd
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 3 deletions.
diff --git a/plugins/cleanupIDs.js b/plugins/cleanupIDs.js
@@ -55,9 +55,24 @@ exports.fn = function(data, params) {
             var item = items.content[i];
 
             // quit if <style> of <script> presents ('force' param prevents quitting)
-            if (!params.force && item.isElem(styleOrScript)) {
-                hasStyleOrScript = true;
-                continue;
+            if (!params.force) {
+                if (item.isElem(styleOrScript)) {
+                    hasStyleOrScript = true;
+                    continue;
+                }
+                // Don't remove IDs if the whole SVG consists only of defs.
+                if (item.isElem('defs') && item.parentNode.isElem('svg')) {
+                    var hasDefsOnly = true;
+                    for (var j = i + 1; j < items.content.length; j++) {
+                        if (items.content[j].isElem()) {
+                            hasDefsOnly = false;
+                            break;
+                        }
+                    }
+                    if (hasDefsOnly) {
+                        break;
+                    }
+                }
             }
             // …and don't remove any ID if yes
             if (item.isElem()) {

diff --git a/test/plugins/cleanupIDs.11.svg b/test/plugins/cleanupIDs.11.svg