diff --git a/packages/twenty-ui/src/navigation/menu/menu-item/components/MenuItemAvatar.tsx b/packages/twenty-ui/src/navigation/menu/menu-item/components/MenuItemAvatar.tsx index c8ec70534c..73d58c74bd 100644 --- a/packages/twenty-ui/src/navigation/menu/menu-item/components/MenuItemAvatar.tsx +++ b/packages/twenty-ui/src/navigation/menu/menu-item/components/MenuItemAvatar.tsx @@ -1,14 +1,7 @@ -import { useTheme } from '@emotion/react'; -import { Avatar, type AvatarProps, IconChevronRight } from '@ui/display'; -import { LightIconButtonGroup } from '@ui/input'; -import { type MenuItemIconButton } from '@ui/navigation/menu/menu-item/components/MenuItem'; -import { MenuItemLeftContent } from '@ui/navigation/menu/menu-item/internals/components/MenuItemLeftContent'; +import { Avatar, type AvatarProps } from '@ui/display'; +import { type MenuItemIconButton, MenuItem } from '@ui/navigation/menu/menu-item/components/MenuItem'; import { type MouseEvent, type ReactNode } from 'react'; import { isDefined } from 'twenty-shared/utils'; -import { - StyledHoverableMenuItemBase, - StyledMenuItemLeftContent, -} from '../internals/components/StyledMenuItemBase'; import { type MenuItemAccent } from '../types/MenuItemAccent'; export type MenuItemAvatarProps = { @@ -30,12 +23,12 @@ export type MenuItemAvatarProps = { contextualText?: ReactNode; }; -// TODO: merge with MenuItem export const MenuItemAvatar = ({ accent = 'default', className, iconButtons, isIconDisplayedOnHoverOnly = true, + isTooltipOpen, onClick, onMouseEnter, onMouseLeave, @@ -45,56 +38,31 @@ export const MenuItemAvatar = ({ text, contextualText, }: MenuItemAvatarProps) => { - const theme = useTheme(); - const showIconButtons = Array.isArray(iconButtons) && iconButtons.length > 0; - - const handleMenuItemClick = (event: MouseEvent) => { - if (!onClick) return; - event.preventDefault(); - event.stopPropagation(); - - onClick?.(event); - }; - return ( - + ) : undefined + } + onClick={onClick} onMouseEnter={onMouseEnter} onMouseLeave={onMouseLeave} - > - - - ) : undefined - } - text={text} - contextualText={contextualText} - /> - -
- {showIconButtons && ( - - )} -
- {hasSubMenu && ( - - )} -
+ testId={testId} + text={text} + contextualText={contextualText} + hasSubMenu={hasSubMenu} + /> ); }; diff --git a/packages/twenty-website/src/app/_components/contributors/PullRequestItem.tsx b/packages/twenty-website/src/app/_components/contributors/PullRequestItem.tsx index dd00e05621..4088301342 100644 --- a/packages/twenty-website/src/app/_components/contributors/PullRequestItem.tsx +++ b/packages/twenty-website/src/app/_components/contributors/PullRequestItem.tsx @@ -1,13 +1,10 @@ -import { Tooltip } from 'react-tooltip'; import styled from '@emotion/styled'; import { format } from 'date-fns'; import { PullRequestIcon } from '@/app/_components/ui/icons/SvgIcons'; import { Theme } from '@/app/_components/ui/theme/theme'; import { formatIntoRelativeDate } from '@/shared-utils/formatIntoRelativeDate'; - -// TODO: use twenty-ui Tooltip -const StyledTooltip = styled(Tooltip)``; +import { AppTooltip } from 'twenty-ui/display'; const Item = styled.div` display: flex; @@ -89,7 +86,8 @@ export const PullRequestItem = ({ #{prNumber} {' '} @@ -97,7 +95,7 @@ export const PullRequestItem = ({ {formatIntoRelativeDate(mergedAt ? mergedAt : createdAt)} - {children}; + return ( + + + {/* Cast for ReactNode type compatibility between Next and Emotion's @types/react */} + {children as any} + + + ); }