fix: only mutate IDs in the output SVG, not the input (#122)

Fixes #85
felixfbecker · Apr 17, 2021 · 7f0ba02 · 7f0ba02
1 parent b6c4cb6
commit 7f0ba02
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/svg.ts b/src/svg.ts
@@ -95,11 +95,6 @@ export function handleSvgElement(element: SVGElement, context: SvgTraversalConte
 			)
 			break
 		}
-
-		// Make all IDs unique
-		for (const descendant of element.querySelectorAll('[id]')) {
-			descendant.id = context.idPrefix + descendant.id
-		}
 	} else {
 		// Clone element
 		if (isSVGAnchorElement(element) && !context.options.keepLinks) {
@@ -155,6 +150,11 @@ export function handleSvgElement(element: SVGElement, context: SvgTraversalConte
 		}
 	}
 
+	// Make sure all IDs are unique
+	if (elementToAppend.id) {
+		elementToAppend.id = context.idPrefix + elementToAppend.id
+	}
+
 	context.currentSvgParent.append(elementToAppend)
 	for (const child of element.childNodes) {
 		handleSvgNode(child, { ...context, currentSvgParent: elementToAppend })