fix: do not throw error when no sprites are found

natemoo-re · Apr 5, 2022 · 3b29d89 · 3b29d89 · vercel · Apr 5, 2022
1 parent b2d4bb8
commit 3b29d89
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/.changeset/gorgeous-pots-drive.md b/.changeset/gorgeous-pots-drive.md
@@ -0,0 +1,5 @@
+---
+"astro-icon": patch
+---
+
+Improve warning when no sprites are found rather than throwing an error
diff --git a/packages/core/lib/context.ts b/packages/core/lib/context.ts
@@ -10,11 +10,15 @@ export function trackSprite(result: any, name: string) {
   }
 }
 
+const warned = new Set();
 export async function getUsedSprites(result: any) {
   if (typeof result[AstroIcon] !== "undefined") {
     return Array.from(result[AstroIcon]["sprites"]);
   }
-  throw new Error(
-    `[astro-icon] <SpriteSheet> should be the very last child of the page's <body>!\nIs it currently placed before any <Sprite> components?`
-  );
+  const pathname = result._metadata.pathname;
+  if (!warned.has(pathname)) {
+    console.log(`[astro-icon] No sprites found while rendering "${pathname}"`);
+    warned.add(pathname);
+  }
+  return []
 }