From 924de4de22e403bae750b87f7da3b4704f989ea3 Mon Sep 17 00:00:00 2001
From: pythongosssss <125205205+pythongosssss@users.noreply.github.com>
Date: Thu, 8 Jan 2026 15:33:47 +0000
Subject: [PATCH] Add deleting progress indicator - add callback to
 confirmDelete - set deleting flag to show progress overlay

---
 .../assets/components/MediaAssetCard.vue         | 16 +++++++++++++++-
 .../assets/components/MediaAssetContextMenu.vue  |  5 ++++-
 .../assets/composables/useMediaAssetActions.ts   | 14 +++++++++++---
 3 files changed, 30 insertions(+), 5 deletions(-)

diff --git a/src/platform/assets/components/MediaAssetCard.vue b/src/platform/assets/components/MediaAssetCard.vue
index ae9671102..15a423bf9 100644
--- a/src/platform/assets/components/MediaAssetCard.vue
+++ b/src/platform/assets/components/MediaAssetCard.vue
@@ -46,6 +46,18 @@
         @image-loaded="handleImageLoaded"
       />
 
+      <div v-if="isDeleting" class="absolute inset-0 bg-black/50 z-10">
+        <div class="flex items-center justify-center h-full relative">
+          <div
+            class="animate-spin rounded-full h-12 w-12 border-t-2 border-b-2 border-white absolute top-1/2 left-1/2 -translate-x-1/2 -translate-y-1/2"
+          ></div>
+          <i
+            class="icon-[lucide--trash-2] size-6 absolute top-1/2 left-1/2 -translate-x-1/2 -translate-y-1/2"
+          >
+          </i>
+        </div>
+      </div>
+
       <!-- Action buttons overlay (top-left) -->
       <div
         v-if="showActionsOverlay"
@@ -129,6 +141,7 @@
     :show-delete-button="showDeleteButton"
     @zoom="handleZoomClick"
     @asset-deleted="emit('asset-deleted')"
+    @asset-deleting="isDeleting = $event"
   />
 </template>
 
@@ -198,6 +211,7 @@ const contextMenu = ref<InstanceType<typeof MediaAssetContextMenu>>()
 
 const isVideoPlaying = ref(false)
 const showVideoControls = ref(false)
+const isDeleting = ref(false)
 
 // Store actual image dimensions
 const imageDimensions = ref<{ width: number; height: number } | undefined>()
@@ -272,7 +286,7 @@ const metaInfo = computed(() => {
 })
 
 const showActionsOverlay = computed(() => {
-  if (loading || !asset) return false
+  if (loading || !asset || isDeleting.value) return false
   return isHovered.value || selected || isVideoPlaying.value
 })
 
diff --git a/src/platform/assets/components/MediaAssetContextMenu.vue b/src/platform/assets/components/MediaAssetContextMenu.vue
index d2b456c16..d10387030 100644
--- a/src/platform/assets/components/MediaAssetContextMenu.vue
+++ b/src/platform/assets/components/MediaAssetContextMenu.vue
@@ -55,6 +55,7 @@ const { asset, assetType, fileKind, showDeleteButton } = defineProps<{
 const emit = defineEmits<{
   zoom: []
   'asset-deleted': []
+  'asset-deleting': [boolean]
 }>()
 
 const contextMenu = ref<InstanceType<typeof ContextMenu>>()
@@ -172,7 +173,9 @@ const contextMenuItems = computed<MenuItem[]>(() => {
       icon: 'icon-[lucide--trash-2]',
       command: async () => {
         if (asset) {
-          const success = await actions.confirmDelete(asset)
+          const success = await actions.confirmDelete(asset, (deleting) => {
+            emit('asset-deleting', deleting)
+          })
           if (success) {
             emit('asset-deleted')
           }
diff --git a/src/platform/assets/composables/useMediaAssetActions.ts b/src/platform/assets/composables/useMediaAssetActions.ts
index ec17364d1..24ac98dfb 100644
--- a/src/platform/assets/composables/useMediaAssetActions.ts
+++ b/src/platform/assets/composables/useMediaAssetActions.ts
@@ -139,7 +139,10 @@ export function useMediaAssetActions() {
    * @param asset The asset to delete
    * @returns true if the asset was deleted, false otherwise
    */
-  const confirmDelete = async (asset: AssetItem): Promise<boolean> => {
+  const confirmDelete = async (
+    asset: AssetItem,
+    onDeleting?: (deleting: boolean) => void
+  ): Promise<boolean> => {
     const assetType = getAssetType(asset)
 
     return new Promise((resolve) => {
@@ -152,8 +155,13 @@ export function useMediaAssetActions() {
           type: 'delete',
           itemList: [asset.name],
           onConfirm: async () => {
-            const success = await deleteAsset(asset, assetType)
-            resolve(success)
+            onDeleting?.(true)
+            try {
+              const success = await deleteAsset(asset, assetType)
+              resolve(success)
+            } finally {
+              onDeleting?.(false)
+            }
           },
           onCancel: () => {
             resolve(false)