feat: enhance icon error handling and loading state management in TotpCodesPage and VaultListIcon components

src/router-public.ts

@@ -126,7 +126,12 @@ function buildConfigResponse(origin: string) {
 }
 
 function normalizeIconHost(rawHost: string): string | null {
-  const decoded = decodeURIComponent(String(rawHost || '').trim()).toLowerCase().replace(/\.+$/, '');
+  let decoded: string;
+  try {
+    decoded = decodeURIComponent(String(rawHost || '').trim()).toLowerCase().replace(/\.+$/, '');
+  } catch {
+    return null;
+  }
   if (!decoded || decoded.includes('/') || decoded.includes('\\')) return null;
   try {
     const parsed = new URL(`https://${decoded}`);

webapp/src/components/TotpCodesPage.tsx

@@ -74,6 +74,18 @@ function TotpListIcon({ cipher }: { cipher: Cipher }) {
   const host = hostFromUri(uri);
   const [errored, setErrored] = useState(() => (host ? failedIconHosts.has(host) : false));
   const [loaded, setLoaded] = useState(false);
+  const markIconError = () => {
+    if (host) failedIconHosts.add(host);
+    setErrored(true);
+  };
+  const syncCachedIconState = (img: HTMLImageElement | null) => {
+    if (!img || !img.complete) return;
+    if (img.naturalWidth > 0) {
+      setLoaded(true);
+      return;
+    }
+    markIconError();
+  };
   useEffect(() => {
     setErrored(host ? failedIconHosts.has(host) : false);
     setLoaded(false);
@@ -91,11 +103,9 @@ function TotpListIcon({ cipher }: { cipher: Cipher }) {
           alt=""
           loading="lazy"
           referrerPolicy="no-referrer"
+          ref={syncCachedIconState}
           onLoad={() => setLoaded(true)}
-          onError={() => {
-            failedIconHosts.add(host);
-            setErrored(true);
-          }}
+          onError={markIconError}
         />
       </span>
     );

webapp/src/components/vault/vault-page-helpers.tsx

@@ -434,6 +434,18 @@ export function VaultListIcon({ cipher }: { cipher: Cipher }) {
   const host = hostFromUri(uri);
   const [errored, setErrored] = useState(() => (host ? failedIconHosts.has(host) : false));
   const [loaded, setLoaded] = useState(false);
+  const markIconError = () => {
+    if (host) failedIconHosts.add(host);
+    setErrored(true);
+  };
+  const syncCachedIconState = (img: HTMLImageElement | null) => {
+    if (!img || !img.complete) return;
+    if (img.naturalWidth > 0) {
+      setLoaded(true);
+      return;
+    }
+    markIconError();
+  };
   useEffect(() => {
     setErrored(host ? failedIconHosts.has(host) : false);
     setLoaded(false);
@@ -443,7 +455,7 @@ export function VaultListIcon({ cipher }: { cipher: Cipher }) {
     return (
       <span className="list-icon-stack">
         <span className={`list-icon-fallback ${loaded ? 'hidden' : ''}`}>
-          <TypeIcon type={Number(cipher.type || 1)} />
+          <Globe size={18} />
         </span>
         <img
           className={`list-icon ${loaded ? 'loaded' : ''}`}
@@ -451,11 +463,9 @@ export function VaultListIcon({ cipher }: { cipher: Cipher }) {
           alt=""
           loading="lazy"
           referrerPolicy="no-referrer"
+          ref={syncCachedIconState}
           onLoad={() => setLoaded(true)}
-          onError={() => {
-            failedIconHosts.add(host);
-            setErrored(true);
-          }}
+          onError={markIconError}
         />
       </span>
     );

webapp/src/styles/vault.css

@@ -376,13 +376,13 @@
   opacity: 1;
 }
 
-.list-icon-fallback.hidden {
-  opacity: 0;
+.list-icon-stack > .list-icon-fallback {
+  opacity: 1;
+  z-index: 1;
 }
 
-.list-icon-stack > .list-icon-fallback {
+.list-icon-stack > .list-icon-fallback.hidden {
   opacity: 0;
-  z-index: 1;
 }
 
 .list-icon-fallback {