fix(web): allow deselecting all assets from select bar (#9320)

* fix(web): allow deselecting all assets from select bar * Change deselect logo * select remove --------- Co-authored-by: Alex <alex.tran1502@gmail.com>
immich-app · May 8, 2024 · 6a5bc15 · 6a5bc15
1 parent 5eaf489
commit 6a5bc15
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/web/src/lib/components/photos-page/actions/select-all-assets.svelte b/web/src/lib/components/photos-page/actions/select-all-assets.svelte
@@ -2,7 +2,7 @@
   import CircleIconButton from '$lib/components/elements/buttons/circle-icon-button.svelte';
   import type { AssetInteractionStore } from '$lib/stores/asset-interaction.store';
   import { type AssetStore, isSelectingAllAssets } from '$lib/stores/assets.store';
-  import { mdiSelectAll, mdiTimerSand } from '@mdi/js';
+  import { mdiSelectAll, mdiSelectRemove } from '@mdi/js';
   import { selectAllAssets } from '$lib/utils/asset-utils';
 
   export let assetStore: AssetStore;
@@ -14,11 +14,12 @@
 
   const handleCancel = () => {
     $isSelectingAllAssets = false;
+    assetInteractionStore.clearMultiselect();
   };
 </script>
 
 {#if $isSelectingAllAssets}
-  <CircleIconButton title="Cancel" icon={mdiTimerSand} on:click={handleCancel} />
+  <CircleIconButton title="Unselect all" icon={mdiSelectRemove} on:click={handleCancel} />
 {:else}
   <CircleIconButton title="Select all" icon={mdiSelectAll} on:click={handleSelectAll} />
 {/if}
diff --git a/web/src/lib/utils/asset-utils.ts b/web/src/lib/utils/asset-utils.ts
@@ -370,7 +370,6 @@ export const selectAllAssets = async (assetStore: AssetStore, assetInteractionSt
     }
   } catch (error) {
     handleError(error, 'Error selecting all assets');
-  } finally {
     isSelectingAllAssets.set(false);
   }
 };