fix: Root URL redirect and HealthIcons display

Aurelius · Aurelius · commit 49ecd671167c · 2026-01-12T00:19:15.000-05:00
- Add index.html to redirect root URL to taskpane.html
- Fix validIcons filter to treat CDN-based icons as always valid
- HealthIcons now shows correct count and Load More button
- Remove duplicate hasCdn declaration
diff --git a/scripts/build.js b/scripts/build.js
@@ -20,7 +20,7 @@ console.log('Building Slides Icons add-in...\n');
 function copyTaskpane() {
     console.log('Copying taskpane files...');
 
-    const files = ['taskpane.html', 'taskpane.css', 'taskpane.js'];
+    const files = ['taskpane.html', 'taskpane.css', 'taskpane.js', 'index.html'];
     const srcDir = path.join(SRC_DIR, 'taskpane');
 
     files.forEach(file => {
diff --git a/src/taskpane/index.html b/src/taskpane/index.html
@@ -0,0 +1,14 @@
+<!DOCTYPE html>
+<html>
+
+<head>
+    <meta charset="UTF-8">
+    <meta http-equiv="refresh" content="0; url=taskpane.html">
+    <title>Redirecting...</title>
+</head>
+
+<body>
+    <p>Redirecting to <a href="taskpane.html">Slides Icons</a>...</p>
+</body>
+
+</html>
diff --git a/src/taskpane/taskpane.js b/src/taskpane/taskpane.js
@@ -157,8 +157,11 @@ function renderIcons() {
         displayLimit = 100;
     }
 
-    // Count valid icons (with actual SVG content)
-    const validIcons = icons.filter(icon => {
+    // Count valid icons (with actual content)
+    // CDN-based manifests: all icons are valid (fetched from CDN)
+    // Embedded SVG manifests: check for valid svg content
+    const hasCdn = !!currentManifest.cdnPattern;
+    const validIcons = hasCdn ? icons : icons.filter(icon => {
         if (typeof icon.svg === 'string') return true;
         if (icon.svg && typeof icon.svg.content === 'string') return true;
         return false;
@@ -183,8 +186,7 @@ function renderIcons() {
         return;
     }
 
-    // Check if manifest uses CDN pattern or embedded SVG
-    const hasCdn = !!currentManifest.cdnPattern;
+    // hasCdn already declared above
 
     grid.innerHTML = displayIcons.map(icon => {
         if (hasCdn) {
