From e19d8bbed771e6d1d63b8a7954feb4d07b517eaa Mon Sep 17 00:00:00 2001
From: Keerat Kohli <kkeerat012@gmail.com>
Date: Thu, 26 Sep 2024 19:49:52 -0400
Subject: [PATCH] Fixes #7220: Remove primary banner icon when only one value
 present

---
 .../meta-types/input/components/EmailsFieldInput.tsx          | 4 +++-
 .../meta-types/input/components/LinksFieldInput.tsx           | 4 +++-
 .../meta-types/input/components/PhonesFieldInput.tsx          | 4 +++-
 3 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/packages/twenty-front/src/modules/object-record/record-field/meta-types/input/components/EmailsFieldInput.tsx b/packages/twenty-front/src/modules/object-record/record-field/meta-types/input/components/EmailsFieldInput.tsx
index 02ce963192b0..32418cebc607 100644
--- a/packages/twenty-front/src/modules/object-record/record-field/meta-types/input/components/EmailsFieldInput.tsx
+++ b/packages/twenty-front/src/modules/object-record/record-field/meta-types/input/components/EmailsFieldInput.tsx
@@ -29,6 +29,8 @@ export const EmailsFieldInput = ({ onCancel }: EmailsFieldInputProps) => {
     });
   };
 
+  const isPrimaryEmail = (index: number) => index === 0 && emails?.length > 1;
+
   return (
     <MultiItemFieldInput
       items={emails}
@@ -46,7 +48,7 @@ export const EmailsFieldInput = ({ onCancel }: EmailsFieldInputProps) => {
         <EmailsFieldMenuItem
           key={index}
           dropdownId={`${hotkeyScope}-emails-${index}`}
-          isPrimary={index === 0}
+          isPrimary={isPrimaryEmail(index)}
           email={email}
           onEdit={handleEdit}
           onSetAsPrimary={handleSetPrimary}
diff --git a/packages/twenty-front/src/modules/object-record/record-field/meta-types/input/components/LinksFieldInput.tsx b/packages/twenty-front/src/modules/object-record/record-field/meta-types/input/components/LinksFieldInput.tsx
index c205039450f0..97bc9578c97c 100644
--- a/packages/twenty-front/src/modules/object-record/record-field/meta-types/input/components/LinksFieldInput.tsx
+++ b/packages/twenty-front/src/modules/object-record/record-field/meta-types/input/components/LinksFieldInput.tsx
@@ -42,6 +42,8 @@ export const LinksFieldInput = ({ onCancel }: LinksFieldInputProps) => {
     });
   };
 
+  const isPrimaryLink = (index: number) => index === 0 && links?.length > 1;
+
   return (
     <MultiItemFieldInput
       items={links}
@@ -61,7 +63,7 @@ export const LinksFieldInput = ({ onCancel }: LinksFieldInputProps) => {
         <LinksFieldMenuItem
           key={index}
           dropdownId={`${hotkeyScope}-links-${index}`}
-          isPrimary={index === 0}
+          isPrimary={isPrimaryLink(index)}
           label={link.label}
           onEdit={handleEdit}
           onSetAsPrimary={handleSetPrimary}
diff --git a/packages/twenty-front/src/modules/object-record/record-field/meta-types/input/components/PhonesFieldInput.tsx b/packages/twenty-front/src/modules/object-record/record-field/meta-types/input/components/PhonesFieldInput.tsx
index 6667acf2031f..54fb7ab8ded9 100644
--- a/packages/twenty-front/src/modules/object-record/record-field/meta-types/input/components/PhonesFieldInput.tsx
+++ b/packages/twenty-front/src/modules/object-record/record-field/meta-types/input/components/PhonesFieldInput.tsx
@@ -78,6 +78,8 @@ export const PhonesFieldInput = ({ onCancel }: PhonesFieldInputProps) => {
     });
   };
 
+  const isPrimaryPhone = (index: number) => index === 0 && phones?.length > 1;
+
   return (
     <MultiItemFieldInput
       items={phones}
@@ -108,7 +110,7 @@ export const PhonesFieldInput = ({ onCancel }: PhonesFieldInputProps) => {
         <PhonesFieldMenuItem
           key={index}
           dropdownId={`${hotkeyScope}-phones-${index}`}
-          isPrimary={index === 0}
+          isPrimary={isPrimaryPhone(index)}
           phone={phone}
           onEdit={handleEdit}
           onSetAsPrimary={handleSetPrimary}