Fix mobile overlay and deps per #43

2025-10-29 11:11:53 +01:00
parent b76e45cb54
commit 69538b86ea
14 changed files with 1396 additions and 450 deletions
@@ -18,11 +18,13 @@ export const AdminAuthenticatedImage: React.FC<AdminAuthenticatedImageProps> = (

  useEffect(() => {
    let cancelled = false;
+    let objectUrl: string | null = null;

    const loadImage = async () => {
      try {
        setLoading(true);
        setError(false);
+        setImageSrc(null);

        // Make authenticated request to get the image
        const response = await api.get(src, {
@@ -31,11 +33,11 @@ export const AdminAuthenticatedImage: React.FC<AdminAuthenticatedImageProps> = (

        if (!cancelled) {
          // Create object URL from blob
-          const imageUrl = URL.createObjectURL(response.data);
-          setImageSrc(imageUrl);
+          objectUrl = URL.createObjectURL(response.data);
+          setImageSrc(objectUrl);
          setLoading(false);
        }
-      } catch (err: any) {
+      } catch {
        // Image loading failed - handled by error state
        if (!cancelled) {
          setError(true);
@@ -51,8 +53,8 @@ export const AdminAuthenticatedImage: React.FC<AdminAuthenticatedImageProps> = (
    // Cleanup function
    return () => {
      cancelled = true;
-      if (imageSrc) {
-        URL.revokeObjectURL(imageSrc);
+      if (objectUrl) {
+        URL.revokeObjectURL(objectUrl);
      }
    };
  }, [src]);
@@ -74,4 +76,4 @@ export const AdminAuthenticatedImage: React.FC<AdminAuthenticatedImageProps> = (
  }

  return <img src={imageSrc || ''} alt={alt} {...props} />;
-};
+};