Skip to content

Commit 2bbdbcc

Browse files
committed
fix(hub-ui): address icon recovery review feedback
1 parent 7a2aab6 commit 2bbdbcc

2 files changed

Lines changed: 5 additions & 4 deletions

File tree

packages/hub-ui/src/client/components/icons/IconifyIcon.vue

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,7 @@ watchEffect(async (onCleanup) => {
4949
</svg>
5050
<div
5151
v-else-if="iconifyParsed"
52+
aria-hidden="true"
5253
v-html="iconifyLoaded"
5354
/>
5455
<img

packages/hub-ui/src/client/utils/iconify.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,10 +28,10 @@ export async function getIconifySvg(collection: string, icon: string) {
2828
if (!response.ok)
2929
throw new Error(`Iconify request failed: ${response.status}`)
3030
const svg = purify.sanitize(await response.text())
31-
const document = new DOMParser().parseFromString(svg, 'image/svg+xml')
32-
if (document.documentElement.localName !== 'svg'
33-
|| document.querySelector('parsererror')
34-
|| !document.querySelector('path, circle, ellipse, rect, line, polyline, polygon, text, use, image')) {
31+
const svgDocument = new DOMParser().parseFromString(svg, 'image/svg+xml')
32+
if (svgDocument.documentElement.localName !== 'svg'
33+
|| svgDocument.querySelector('parsererror')
34+
|| !svgDocument.querySelector('path, circle, ellipse, rect, line, polyline, polygon, text, use, image')) {
3535
throw new Error('Iconify returned an invalid SVG')
3636
}
3737
return svg

0 commit comments

Comments
 (0)