From eab3a8b6139a01dde44669fe7095ef961cdd4426 Mon Sep 17 00:00:00 2001 From: Yulia Goldberg Date: Mon, 29 Jan 2024 12:16:03 +0200 Subject: [PATCH] fix(MenuItem): split menu items does not open sub-menu when navigating directly between two split menus (#1904) --- .../Menu/MenuItem/hooks/useMenuItemMouseEvents.ts | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/components/Menu/MenuItem/hooks/useMenuItemMouseEvents.ts b/src/components/Menu/MenuItem/hooks/useMenuItemMouseEvents.ts index 12bf08bcdb..3cca8ad5c4 100644 --- a/src/components/Menu/MenuItem/hooks/useMenuItemMouseEvents.ts +++ b/src/components/Menu/MenuItem/hooks/useMenuItemMouseEvents.ts @@ -40,16 +40,11 @@ export default function useMenuItemMouseEvents({ } }, [ setSubMenuIsOpenByIndex, - splitMenuItemIconButtonRef, index, - hasChildren, - splitMenuItem, isMouseEnterOnIconButton, prevIsMouseEnterOnIconButton, isActive, - resetOpenSubMenuIndex, - setActiveItemIndex, - isMouseEnter + resetOpenSubMenuIndex ]); useLayoutEffect(() => { @@ -78,11 +73,16 @@ export default function useMenuItemMouseEvents({ if (!isActive && splitMenuItem) { setActiveItemIndex(index); + + if (isMouseEnterOnIconButton) { + setSubMenuIsOpenByIndex(index, true); + } } }, [ resetOpenSubMenuIndex, prevIsMouseEnter, isMouseEnter, + isMouseEnterOnIconButton, setSubMenuIsOpenByIndex, isActive, setActiveItemIndex,