Don’t scroll menus in response to hover (#2451)

By default, when a menu item is focused, browsers will scroll it into view.
Sometimes even into the centre of the view. This is good when using a keyboard,
because the focused item is automatically visible, but when it was focused in
response to mouse movement, it has the effect of moving the item out from
underneath the pointer.

Fixes #1566

.yarn/versions/54d1ffa8.yml

@@ -0,0 +1,8 @@
+releases:
+  "@radix-ui/react-context-menu": patch
+  "@radix-ui/react-dropdown-menu": patch
+  "@radix-ui/react-menu": patch
+  "@radix-ui/react-menubar": patch
+
+declined:
+  - primitives

packages/react/menu/src/Menu.tsx

@@ -729,7 +729,7 @@ const MenuItemImpl = React.forwardRef<MenuItemImplElement, MenuItemImplProps>(
                   contentContext.onItemEnter(event);
                   if (!event.defaultPrevented) {
                     const item = event.currentTarget;
-                    item.focus();
+                    item.focus({ preventScroll: true });
                   }
                 }
               })