Use first item of few with same ID (as browsers do). Remove subsequen…

…t IDs. Resolves svg#378
megomars · Jun 21, 2015 · 2c2f9e7 · 2c2f9e7
1 parent 20e2a30
commit 2c2f9e7
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/plugins/cleanupIDs.js b/plugins/cleanupIDs.js
@@ -67,7 +67,11 @@ exports.fn = function(data, params) {
                     item.eachAttr(function(attr) {
                         // save IDs
                         if (attr.name === 'id') {
-                            IDs[idPrefix + attr.value] = item;
+                            if (idPrefix + attr.value in IDs) {
+                                item.removeAttr('id');
+                            } else {
+                                IDs[idPrefix + attr.value] = item;
+                            }
                         }
 
                         // save IDs url() references

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