From dfc17043fb1f0aa374344b57f8ad186aebda93cc Mon Sep 17 00:00:00 2001 From: Nate Kelley Date: Wed, 1 Oct 2025 09:42:33 -0600 Subject: [PATCH] hover should trigger selected, not hover --- .../ui/inputs/MentionInput/MentionList/MentionListItem.tsx | 7 ++++--- .../MentionInput/MentionList/MentionListSelector.tsx | 1 + 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/apps/web/src/components/ui/inputs/MentionInput/MentionList/MentionListItem.tsx b/apps/web/src/components/ui/inputs/MentionInput/MentionList/MentionListItem.tsx index 77d9de047..f8f00e186 100644 --- a/apps/web/src/components/ui/inputs/MentionInput/MentionList/MentionListItem.tsx +++ b/apps/web/src/components/ui/inputs/MentionInput/MentionList/MentionListItem.tsx @@ -13,7 +13,8 @@ export function MentionListItem({ disabled, onSelectItem, secondaryContent, -}: Omit, 'setSelectedItem'>) { + setSelectedItem, +}: MentionTriggerItemExtended) { const containerRef = useRef(null); const listRef = useMentionListRef(); @@ -39,9 +40,9 @@ export function MentionListItem({ 'group/mention-list-item', 'flex items-center justify-between gap-x-1.5 overflow-hidden', `cursor-pointer px-2.5 min-h-8 text-base rounded transition-all duration-100`, - 'hover:bg-item-hover', - isSelected && 'bg-item-hover' + 'data-[selected=true]:bg-item-hover' )} + onMouseEnter={() => setSelectedItem(value)} >
{icon && {icon}} diff --git a/apps/web/src/components/ui/inputs/MentionInput/MentionList/MentionListSelector.tsx b/apps/web/src/components/ui/inputs/MentionInput/MentionList/MentionListSelector.tsx index 76a15940b..7b3b47c95 100644 --- a/apps/web/src/components/ui/inputs/MentionInput/MentionList/MentionListSelector.tsx +++ b/apps/web/src/components/ui/inputs/MentionInput/MentionList/MentionListSelector.tsx @@ -40,6 +40,7 @@ export function MentionListSelector(props: MentionInputTriggerItemEx } const _exhaustiveCheck: never = type; + console.warn('mention list selector type not found', _exhaustiveCheck); return null; }