From 1fadbda6d2d594151ae0d92dd178ca3a35d4d7e7 Mon Sep 17 00:00:00 2001
From: Sam Becker <sam@sambecker.com>
Date: Sat, 22 Mar 2025 17:37:03 -0500
Subject: [PATCH] Refine upload status behavior

---
 src/admin/upload/AdminUploadPanel.tsx |  5 ++++-
 src/photo/PhotoUploadWithStatus.tsx   | 31 ++++++++++++++++-----------
 2 files changed, 23 insertions(+), 13 deletions(-)

diff --git a/src/admin/upload/AdminUploadPanel.tsx b/src/admin/upload/AdminUploadPanel.tsx
index 2a3fa272..5d809f06 100644
--- a/src/admin/upload/AdminUploadPanel.tsx
+++ b/src/admin/upload/AdminUploadPanel.tsx
@@ -20,13 +20,16 @@ export default function AdminUploadPanel({
     uploadState: {
       isUploading,
       hideUploadPanel,
+      uploadError,
     },
     resetUploadState,
   } = useAppState();
 
   return (
     <AppGrid
-      className={clsx((!isUploading || hideUploadPanel) && 'hidden')}
+      className={clsx(
+        ((!isUploading || uploadError) || hideUploadPanel) && 'hidden',
+      )}
       contentMain={
         <Container
           color="gray"
diff --git a/src/photo/PhotoUploadWithStatus.tsx b/src/photo/PhotoUploadWithStatus.tsx
index 1fd8ff38..0b43a919 100644
--- a/src/photo/PhotoUploadWithStatus.tsx
+++ b/src/photo/PhotoUploadWithStatus.tsx
@@ -74,7 +74,9 @@ export default function PhotoUploadWithStatus({
   }, [resetUploadState]);
   const isFinishing = isPending && shouldResetUploadStateAfterPending.current;
 
-  const uploadNumberText = `${fileUploadIndex + 1} of ${filesLength}`;
+  const uploadStatusText = filesLength > 1
+    ? `${fileUploadIndex + 1} of ${filesLength}`
+    : undefined;
 
   return (
     <div className={clsx(
@@ -127,6 +129,7 @@ export default function PhotoUploadWithStatus({
                   }
                 })
                 .catch(error => {
+                  console.error(error);
                   setUploadState?.({
                     isUploading: false,
                     uploadError: `Upload Error: ${error.message}`,
@@ -155,17 +158,25 @@ export default function PhotoUploadWithStatus({
                 Finishing ...
               </>
               : <>
-                {!showButton && <>
-                  <ResponsiveText shortText={uploadNumberText}>
-                    Uploading {uploadNumberText}
-                  </ResponsiveText>
-                  {': '}
-                </>}
-                {fileUploadName}
+                {!showButton && uploadStatusText
+                  ? <>
+                    <ResponsiveText shortText={uploadStatusText}>
+                      Uploading {uploadStatusText}
+                    </ResponsiveText>
+                    {': '}
+                    {fileUploadName}
+                  </>
+                  : <ResponsiveText shortText={fileUploadName}>
+                    Uploading {fileUploadName}
+                  </ResponsiveText>}
               </>
             : !showButton && <>Initializing</>}
         </span>
       </div>}
+      {uploadError &&
+        <div className="text-error">
+          {uploadError}
+        </div>}
       {debug && debugDownload &&
         <a
           className="block"
@@ -174,10 +185,6 @@ export default function PhotoUploadWithStatus({
         >
           Download
         </a>}
-      {uploadError &&
-        <div className="text-error">
-          {uploadError}
-        </div>}
     </div>
   );
 };