From 96aa17e10b9aca8ab67ae8d0131569d6f436cd32 Mon Sep 17 00:00:00 2001
From: Liam DeBeasi <liamdebeasi@users.noreply.github.com>
Date: Wed, 1 Nov 2023 10:08:52 -0400
Subject: [PATCH] fix(icon): add better warning when loading icons

---
 src/components/icon/utils.ts | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/components/icon/utils.ts b/src/components/icon/utils.ts
index f6c664479..16206eab4 100644
--- a/src/components/icon/utils.ts
+++ b/src/components/icon/utils.ts
@@ -88,7 +88,16 @@ const getNamedUrl = (iconName: string) => {
   if (url) {
     return url;
   }
-  return getAssetPath(`svg/${iconName}.svg`);
+  try {
+    return getAssetPath(`svg/${iconName}.svg`);
+  } catch(e) {
+    /**
+     * In the custom elements build version of ionicons, referencing an icon
+     * by name will throw an invalid URL error because the asset path is not defined.
+     * This catches that error and logs something that is more developer-friendly.
+     */
+    console.warn(`[Ionicons Warning]: Could not load icon with name "${name}". Ensure that the icon is registered using addIcons or that the icon SVG data is passed directly to the icon component.`);
+  }
 };