From 411772123f1023339f1014e9c66206c0f2582251 Mon Sep 17 00:00:00 2001
From: Ben McCann <322311+benmccann@users.noreply.github.com>
Date: Thu, 27 Mar 2025 15:12:14 -0700
Subject: [PATCH] chore(web): remove unused props (#17141)

---
 .../notification-settings/notification-settings.svelte       | 2 +-
 .../settings/template-settings/template-settings.svelte      | 5 -----
 web/src/lib/components/asset-viewer/asset-viewer.svelte      | 2 --
 web/src/lib/components/asset-viewer/photo-viewer.svelte      | 1 -
 .../lib/components/assets/thumbnail/image-thumbnail.svelte   | 2 --
 .../components/faces-page/manage-people-visibility.svelte    | 3 +--
 .../lib/components/faces-page/merge-suggestion-modal.svelte  | 1 -
 web/src/lib/components/faces-page/people-card.svelte         | 4 +---
 .../components/shared-components/scrubber/scrubber.svelte    | 1 -
 .../shared-components/side-bar/side-bar-link.svelte          | 1 -
 web/src/routes/(user)/people/+page.svelte                    | 3 +--
 11 files changed, 4 insertions(+), 21 deletions(-)

diff --git a/web/src/lib/components/admin-page/settings/notification-settings/notification-settings.svelte b/web/src/lib/components/admin-page/settings/notification-settings/notification-settings.svelte
index aa4520536a..a2b6305f76 100644
--- a/web/src/lib/components/admin-page/settings/notification-settings/notification-settings.svelte
+++ b/web/src/lib/components/admin-page/settings/notification-settings/notification-settings.svelte
@@ -170,7 +170,7 @@
       </div>
     </form>
   </div>
-  <TemplateSettings {defaultConfig} {config} {savedConfig} {onReset} {onSave} />
+  <TemplateSettings {config} {savedConfig} />
 
   <SettingButtonsRow
     onReset={(options) => onReset({ ...options, configKeys: ['notifications', 'templates'] })}
diff --git a/web/src/lib/components/admin-page/settings/template-settings/template-settings.svelte b/web/src/lib/components/admin-page/settings/template-settings/template-settings.svelte
index eda1d8e091..b418aebb2b 100644
--- a/web/src/lib/components/admin-page/settings/template-settings/template-settings.svelte
+++ b/web/src/lib/components/admin-page/settings/template-settings/template-settings.svelte
@@ -11,15 +11,10 @@
   import { mdiEyeOutline } from '@mdi/js';
   import { t } from 'svelte-i18n';
   import { fade } from 'svelte/transition';
-  import type { SettingsResetEvent, SettingsSaveEvent } from '../admin-settings';
 
   interface Props {
     savedConfig: SystemConfigDto;
-    defaultConfig: SystemConfigDto;
     config: SystemConfigDto;
-    disabled?: boolean;
-    onReset: SettingsResetEvent;
-    onSave: SettingsSaveEvent;
   }
 
   let { savedConfig, config = $bindable() }: Props = $props();
diff --git a/web/src/lib/components/asset-viewer/asset-viewer.svelte b/web/src/lib/components/asset-viewer/asset-viewer.svelte
index dec96f92f8..00d526353e 100644
--- a/web/src/lib/components/asset-viewer/asset-viewer.svelte
+++ b/web/src/lib/components/asset-viewer/asset-viewer.svelte
@@ -482,7 +482,6 @@
             {preloadAssets}
             onPreviousAsset={() => navigateAsset('previous')}
             onNextAsset={() => navigateAsset('next')}
-            onClose={closeViewer}
             haveFadeTransition={false}
             {sharedLink}
           />
@@ -527,7 +526,6 @@
               {preloadAssets}
               onPreviousAsset={() => navigateAsset('previous')}
               onNextAsset={() => navigateAsset('next')}
-              onClose={closeViewer}
               {sharedLink}
               haveFadeTransition={$slideshowState === SlideshowState.None || $slideshowTransition}
             />
diff --git a/web/src/lib/components/asset-viewer/photo-viewer.svelte b/web/src/lib/components/asset-viewer/photo-viewer.svelte
index 70467ccb82..754693897a 100644
--- a/web/src/lib/components/asset-viewer/photo-viewer.svelte
+++ b/web/src/lib/components/asset-viewer/photo-viewer.svelte
@@ -32,7 +32,6 @@
     onNextAsset?: (() => void) | null;
     copyImage?: () => Promise<void>;
     zoomToggle?: (() => void) | null;
-    onClose?: () => void;
   }
 
   let {
diff --git a/web/src/lib/components/assets/thumbnail/image-thumbnail.svelte b/web/src/lib/components/assets/thumbnail/image-thumbnail.svelte
index 119efe71b5..6c0487eb60 100644
--- a/web/src/lib/components/assets/thumbnail/image-thumbnail.svelte
+++ b/web/src/lib/components/assets/thumbnail/image-thumbnail.svelte
@@ -18,10 +18,8 @@
     circle?: boolean;
     hidden?: boolean;
     border?: boolean;
-    preload?: boolean;
     hiddenIconClass?: string;
     onComplete?: (() => void) | undefined;
-    onClick?: (() => void) | undefined;
   }
 
   let {
diff --git a/web/src/lib/components/faces-page/manage-people-visibility.svelte b/web/src/lib/components/faces-page/manage-people-visibility.svelte
index 196fe13900..0c2ed7bad0 100644
--- a/web/src/lib/components/faces-page/manage-people-visibility.svelte
+++ b/web/src/lib/components/faces-page/manage-people-visibility.svelte
@@ -136,7 +136,7 @@
 
 <div class="flex flex-wrap gap-1 bg-immich-bg p-2 pb-8 dark:bg-immich-dark-bg md:px-8 mt-16">
   <PeopleInfiniteScroll {people} hasNextPage={true} {loadNextPage}>
-    {#snippet children({ person, index })}
+    {#snippet children({ person })}
       {@const hidden = personIsHidden[person.id]}
       <button
         type="button"
@@ -146,7 +146,6 @@
         aria-label={person.name ? $t('hide_named_person', { values: { name: person.name } }) : $t('hide_person')}
       >
         <ImageThumbnail
-          preload={index < 20}
           {hidden}
           shadow
           url={getPeopleThumbnailUrl(person)}
diff --git a/web/src/lib/components/faces-page/merge-suggestion-modal.svelte b/web/src/lib/components/faces-page/merge-suggestion-modal.svelte
index eaf9ab64e9..d4f8a6f416 100644
--- a/web/src/lib/components/faces-page/merge-suggestion-modal.svelte
+++ b/web/src/lib/components/faces-page/merge-suggestion-modal.svelte
@@ -100,7 +100,6 @@
                     url={getPeopleThumbnailUrl(person)}
                     altText={person.name}
                     widthStyle="100%"
-                    onClick={() => changePersonToMerge(person)}
                   />
                 </button>
               </div>
diff --git a/web/src/lib/components/faces-page/people-card.svelte b/web/src/lib/components/faces-page/people-card.svelte
index d12855c54f..4aff4e96f8 100644
--- a/web/src/lib/components/faces-page/people-card.svelte
+++ b/web/src/lib/components/faces-page/people-card.svelte
@@ -20,14 +20,13 @@
 
   interface Props {
     person: PersonResponseDto;
-    preload?: boolean;
     onSetBirthDate: () => void;
     onMergePeople: () => void;
     onHidePerson: () => void;
     onToggleFavorite: () => void;
   }
 
-  let { person, preload = false, onSetBirthDate, onMergePeople, onHidePerson, onToggleFavorite }: Props = $props();
+  let { person, onSetBirthDate, onMergePeople, onHidePerson, onToggleFavorite }: Props = $props();
 
   let showVerticalDots = $state(false);
 </script>
@@ -48,7 +47,6 @@
     <div class="w-full h-full rounded-xl brightness-95 filter">
       <ImageThumbnail
         shadow
-        {preload}
         url={getPeopleThumbnailUrl(person)}
         altText={person.name}
         title={person.name}
diff --git a/web/src/lib/components/shared-components/scrubber/scrubber.svelte b/web/src/lib/components/shared-components/scrubber/scrubber.svelte
index 0fdfa9c0da..581e068666 100644
--- a/web/src/lib/components/shared-components/scrubber/scrubber.svelte
+++ b/web/src/lib/components/shared-components/scrubber/scrubber.svelte
@@ -14,7 +14,6 @@
     timelineBottomOffset?: number;
     height?: number;
     assetStore: AssetStore;
-    invisible?: boolean;
     scrubOverallPercent?: number;
     scrubBucketPercent?: number;
     scrubBucket?: { bucketDate: string | undefined };
diff --git a/web/src/lib/components/shared-components/side-bar/side-bar-link.svelte b/web/src/lib/components/shared-components/side-bar/side-bar-link.svelte
index c7f8cd4fae..47de0d5db0 100644
--- a/web/src/lib/components/shared-components/side-bar/side-bar-link.svelte
+++ b/web/src/lib/components/shared-components/side-bar/side-bar-link.svelte
@@ -13,7 +13,6 @@
     flippedLogo?: boolean;
     isSelected?: boolean;
     preloadData?: boolean;
-    moreInformation?: Snippet;
     dropDownContent?: Snippet;
     dropdownOpen?: boolean;
   }
diff --git a/web/src/routes/(user)/people/+page.svelte b/web/src/routes/(user)/people/+page.svelte
index 45e18fd398..3e1b5a774c 100644
--- a/web/src/routes/(user)/people/+page.svelte
+++ b/web/src/routes/(user)/people/+page.svelte
@@ -408,13 +408,12 @@
 
   {#if countVisiblePeople > 0 && (!searchName || searchedPeopleLocal.length > 0)}
     <PeopleInfiniteScroll people={showPeople} hasNextPage={!!nextPage && !searchName} {loadNextPage}>
-      {#snippet children({ person, index })}
+      {#snippet children({ person })}
         <div
           class="p-2 rounded-xl hover:bg-gray-200 border-2 hover:border-immich-primary/50 hover:shadow-sm dark:hover:bg-immich-dark-primary/20 hover:dark:border-immich-dark-primary/25 border-transparent transition-all"
         >
           <PeopleCard
             {person}
-            preload={index < 20}
             onSetBirthDate={() => handleSetBirthDate(person)}
             onMergePeople={() => handleMergePeople(person)}
             onHidePerson={() => handleHidePerson(person)}