From 1e402aca5fe5b9daa0289c15dc4bf9d68bdd8a46 Mon Sep 17 00:00:00 2001
From: brendanlaschke <brendan.laschke@googlemail.com>
Date: Wed, 4 Oct 2023 15:49:25 +0200
Subject: [PATCH] Remove company avatar show page hover effect (#1857)

- remove company avatar show page hover effect
---
 .../ui/layout/show-page/components/ShowPageSummaryCard.tsx    | 4 ++--
 front/src/modules/users/components/Avatar.tsx                 | 3 ++-
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/front/src/modules/ui/layout/show-page/components/ShowPageSummaryCard.tsx b/front/src/modules/ui/layout/show-page/components/ShowPageSummaryCard.tsx
index 3ceb9c0a5..8ba72d6ba 100644
--- a/front/src/modules/ui/layout/show-page/components/ShowPageSummaryCard.tsx
+++ b/front/src/modules/ui/layout/show-page/components/ShowPageSummaryCard.tsx
@@ -92,7 +92,7 @@ export const ShowPageSummaryCard = ({
     if (e.target.files) onUploadPicture?.(e.target.files[0]);
   };
   const handleAvatarClick = () => {
-    if (onUploadPicture) inputFileRef?.current?.click?.();
+    inputFileRef?.current?.click?.();
   };
 
   return (
@@ -100,7 +100,7 @@ export const ShowPageSummaryCard = ({
       <StyledAvatarWrapper>
         <Avatar
           avatarUrl={logoOrAvatar}
-          onClick={handleAvatarClick}
+          onClick={onUploadPicture ? handleAvatarClick : undefined}
           size="xl"
           colorId={id}
           placeholder={title}
diff --git a/front/src/modules/users/components/Avatar.tsx b/front/src/modules/users/components/Avatar.tsx
index a285dd1e1..3ee8aca28 100644
--- a/front/src/modules/users/components/Avatar.tsx
+++ b/front/src/modules/users/components/Avatar.tsx
@@ -83,7 +83,8 @@ const StyledAvatar = styled.div<AvatarProps & { colorId: string }>`
   }};
 
   &:hover {
-    box-shadow: 0 0 0 4px ${({ theme }) => theme.background.transparent.light};
+    box-shadow: ${({ theme, onClick }) =>
+      onClick ? '0 0 0 4px ' + theme.background.transparent.light : 'unset'};
   }
 `;