From 94585b4a7f5cd233a4547db69cd00efb9fa0609f Mon Sep 17 00:00:00 2001
From: Sam Becker <sam@sambecker.com>
Date: Sun, 21 Jul 2024 10:58:18 -0500
Subject: [PATCH] Fix multi-photo delete loading status

---
 src/admin/AdminBatchEditPanelClient.tsx |  2 ++
 src/admin/DeletePhotosButton.tsx        | 10 +++++++++-
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/src/admin/AdminBatchEditPanelClient.tsx b/src/admin/AdminBatchEditPanelClient.tsx
index 8f3c8d0a..91a07302 100644
--- a/src/admin/AdminBatchEditPanelClient.tsx
+++ b/src/admin/AdminBatchEditPanelClient.tsx
@@ -100,7 +100,9 @@ export default function AdminBatchEditPanelClient({
           <DeletePhotosButton
             photoIds={selectedPhotoIds}
             disabled={isPerformingSelectEdit}
+            onClick={() => setIsPerformingSelectEdit?.(true)}
             onDelete={resetForm}
+            onFinish={() => setIsPerformingSelectEdit?.(false)}
           />
         </>}
       <LoaderButton
diff --git a/src/admin/DeletePhotosButton.tsx b/src/admin/DeletePhotosButton.tsx
index 67422d4a..68492f4a 100644
--- a/src/admin/DeletePhotosButton.tsx
+++ b/src/admin/DeletePhotosButton.tsx
@@ -14,11 +14,15 @@ export default function DeletePhotosButton({
   onDelete,
   clearLocalState = true,
   className,
+  onClick,
+  onFinish,
   confirmText,
   toastText,
   ...rest
 }: {
   photoIds?: string[]
+  onClick?: () => void
+  onFinish?: () => void
   onDelete?: () => void
   clearLocalState?: boolean
   toastText?: string
@@ -47,6 +51,7 @@ export default function DeletePhotosButton({
       // eslint-disable-next-line max-len
       confirmText={confirmText ?? `Are you sure you want to delete ${photosText}? This action cannot be undone.`}
       onClick={() => {
+        onClick?.();
         setIsLoading(true);
         deletePhotosAction(photoIds)
           .then(() => {
@@ -58,7 +63,10 @@ export default function DeletePhotosButton({
             onDelete?.();
           })
           .catch(() => toastWarning(`Failed to delete ${photosText}`))
-          .finally(() => setIsLoading(false));
+          .finally(() => {
+            setIsLoading(false);
+            onFinish?.();
+          });
       }}
     />
   );