fix(MenuItem): split menu items does not open sub-menu when navigatin…

…g directly between two split menus (#1904)
mondaycom · Jan 29, 2024 · eab3a8b · eab3a8b
1 parent 61a50dd
commit eab3a8b
Showing 1 changed file with 6 additions and 6 deletions.
diff --git 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,