fix(): use DOMParser with "image/svg+xml" to parse svg

Author: weareoutman

bricks/icons/src/shared/SvgCache.ts

@@ -29,18 +29,15 @@ export function constructSvgElement(
   retryable: boolean,
   options?: ResolveIconOptions
 ): SVGResult | null {
-  const div = document.createElement("div");
-  div.innerHTML = content;
-
-  const svg = div.firstElementChild;
-  if (svg?.tagName?.toLowerCase() !== "svg")
-    return retryable ? CACHEABLE_ERROR : null;
-
   if (!parser) parser = new DOMParser();
-  const doc = parser.parseFromString(svg.outerHTML, "text/html");
-
-  const svgEl = doc.body.querySelector("svg");
-  if (!svgEl) return retryable ? CACHEABLE_ERROR : null;
+  // Requires `xmlns`for the SVG to be parsed correctly
+  const refinedContent = content.includes("xmlns=")
+    ? content
+    : content.replace(/(<svg)(\s)/i, '$1 xmlns="http://www.w3.org/2000/svg"$2');
+  const doc = parser.parseFromString(refinedContent, "image/svg+xml");
+  const svgEl = doc.documentElement;
+  if (!(svgEl instanceof SVGSVGElement))
+    return retryable ? CACHEABLE_ERROR : null;
 
   const titles = svgEl.querySelectorAll("title");
   for (const title of titles) {