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 a0a1a4608..d1a3f3bf6 100644 --- a/front/src/modules/ui/layout/show-page/components/ShowPageSummaryCard.tsx +++ b/front/src/modules/ui/layout/show-page/components/ShowPageSummaryCard.tsx @@ -89,15 +89,16 @@ export const ShowPageSummaryCard = ({ const onFileChange = (e: ChangeEvent) => { if (e.target.files) onUploadPicture?.(e.target.files[0]); }; - const onAvatarClick = () => { + const handleAvatarClick = () => { if (onUploadPicture) inputFileRef?.current?.click?.(); }; return ( - + theme.spacing(2)}; position: relative; `; - export type UpdateViewButtonGroupProps = { hotkeyScope: string; onViewEditModeChange?: () => void; diff --git a/front/src/modules/users/components/Avatar.tsx b/front/src/modules/users/components/Avatar.tsx index 844912475..1b9f0e7a8 100644 --- a/front/src/modules/users/components/Avatar.tsx +++ b/front/src/modules/users/components/Avatar.tsx @@ -16,6 +16,7 @@ export type AvatarProps = { placeholder: string; colorId?: string; type?: AvatarType; + onClick?: () => void; }; const StyledAvatar = styled.div` @@ -28,6 +29,7 @@ const StyledAvatar = styled.div` background-size: cover; border-radius: ${(props) => (props.type === 'rounded' ? '50%' : '2px')}; color: ${({ colorId }) => stringToHslColor(colorId, 75, 25)}; + cursor: ${({ onClick }) => (onClick ? 'pointer' : 'default')}; display: flex; flex-shrink: 0; @@ -79,6 +81,10 @@ const StyledAvatar = styled.div` return '12px'; } }}; + + &:hover { + box-shadow: 0 0 0 4px ${theme.background.transparent.light}; + } `; export const Avatar = ({ @@ -86,6 +92,7 @@ export const Avatar = ({ size = 'md', placeholder, colorId = placeholder, + onClick, type = 'squared', }: AvatarProps) => { const noAvatarUrl = !isNonEmptyString(avatarUrl); @@ -111,6 +118,7 @@ export const Avatar = ({ size={size} type={type} colorId={colorId} + onClick={onClick} > {(noAvatarUrl || isInvalidAvatarUrl) && placeholder[0]?.toLocaleUpperCase()}