From c44ea24288cfa807b87b996c3fd9844c8722bc9b Mon Sep 17 00:00:00 2001
From: Ben Basten <45583362+ben-basten@users.noreply.github.com>
Date: Sat, 15 Jun 2024 08:09:01 -0400
Subject: [PATCH] fix: change redundant prop naming

---
 .../components/album-page/share-info-modal.svelte    |  2 +-
 .../asset-viewer/asset-viewer-nav-bar.svelte         |  8 +-------
 web/src/lib/components/faces-page/people-card.svelte |  6 +++---
 .../context-menu/button-context-menu.svelte          | 12 ++++++------
 web/src/routes/admin/library-management/+page.svelte |  4 ++--
 5 files changed, 13 insertions(+), 19 deletions(-)

diff --git a/web/src/lib/components/album-page/share-info-modal.svelte b/web/src/lib/components/album-page/share-info-modal.svelte
index 0fefc441686ea4..82403efcbe21b8 100644
--- a/web/src/lib/components/album-page/share-info-modal.svelte
+++ b/web/src/lib/components/album-page/share-info-modal.svelte
@@ -107,7 +107,7 @@
               {/if}
             </div>
             {#if isOwned}
-              <ButtonContextMenu icon={mdiDotsVertical} buttonSize="20" title={$t('options')}>
+              <ButtonContextMenu icon={mdiDotsVertical} size="20" title={$t('options')}>
                 {#if role === AlbumUserRole.Viewer}
                   <MenuOption on:click={() => handleSetReadonly(user, AlbumUserRole.Editor)} text={$t('allow_edits')} />
                 {:else}
diff --git a/web/src/lib/components/asset-viewer/asset-viewer-nav-bar.svelte b/web/src/lib/components/asset-viewer/asset-viewer-nav-bar.svelte
index 1acae5814b1605..76a6494c824152 100644
--- a/web/src/lib/components/asset-viewer/asset-viewer-nav-bar.svelte
+++ b/web/src/lib/components/asset-viewer/asset-viewer-nav-bar.svelte
@@ -175,13 +175,7 @@
         on:delete={() => dispatch('delete')}
         on:permanentlyDelete={() => dispatch('permanentlyDelete')}
       />
-      <ButtonContextMenu
-        direction="left"
-        align="top-right"
-        buttonColor="opaque"
-        title={$t('more')}
-        icon={mdiDotsVertical}
-      >
+      <ButtonContextMenu direction="left" align="top-right" color="opaque" title={$t('more')} icon={mdiDotsVertical}>
         {#if showSlideshow}
           <MenuOption icon={mdiPresentationPlay} on:click={() => onMenuClick('playSlideShow')} text={$t('slideshow')} />
         {/if}
diff --git a/web/src/lib/components/faces-page/people-card.svelte b/web/src/lib/components/faces-page/people-card.svelte
index c80cb65e775695..5585576283d3ab 100644
--- a/web/src/lib/components/faces-page/people-card.svelte
+++ b/web/src/lib/components/faces-page/people-card.svelte
@@ -69,9 +69,9 @@
   <div class="absolute top-2 right-2">
     <ButtonContextMenu
       buttonClass="icon-white-drop-shadow focus:opacity-100 {showVerticalDots ? 'opacity-100' : 'opacity-0'}"
-      buttonColor="opaque"
-      buttonPadding="2"
-      buttonSize="20"
+      color="opaque"
+      padding="2"
+      size="20"
       icon={mdiDotsVertical}
       title={$t('show_person_options')}
       usePortal
diff --git a/web/src/lib/components/shared-components/context-menu/button-context-menu.svelte b/web/src/lib/components/shared-components/context-menu/button-context-menu.svelte
index 0ebf5c62aeec15..66a99c0fcbca75 100644
--- a/web/src/lib/components/shared-components/context-menu/button-context-menu.svelte
+++ b/web/src/lib/components/shared-components/context-menu/button-context-menu.svelte
@@ -19,9 +19,9 @@
    * The direction in which the context menu should open.
    */
   export let direction: 'left' | 'right' = 'right';
-  export let buttonColor: Color = 'transparent';
-  export let buttonSize: string | undefined = undefined;
-  export let buttonPadding: string | undefined = undefined;
+  export let color: Color = 'transparent';
+  export let size: string | undefined = undefined;
+  export let padding: string | undefined = undefined;
   /**
    * Additional classes to apply to the button.
    */
@@ -93,17 +93,17 @@
     bind:this={buttonContainer}
   >
     <CircleIconButton
+      {color}
       {icon}
+      {padding}
+      {size}
       {title}
       ariaControls={menuId}
       ariaExpanded={showContextMenu}
       ariaHasPopup={true}
       class={buttonClass}
-      color={buttonColor}
       id={buttonId}
       on:click={handleClick}
-      padding={buttonPadding}
-      size={buttonSize}
     />
   </div>
   {#if usePortal}
diff --git a/web/src/routes/admin/library-management/+page.svelte b/web/src/routes/admin/library-management/+page.svelte
index d9cec07201c4a3..f6bdd6ad0a8d98 100644
--- a/web/src/routes/admin/library-management/+page.svelte
+++ b/web/src/routes/admin/library-management/+page.svelte
@@ -374,8 +374,8 @@
                 <td class=" text-ellipsis px-4 text-sm">
                   <ButtonContextMenu
                     align="middle"
-                    buttonColor="primary"
-                    buttonSize="16"
+                    color="primary"
+                    size="16"
                     icon={mdiDotsVertical}
                     title={$t('library_options')}
                     usePortal