Opened 9 years ago

Closed 6 years ago

#10979 closed enhancement (fixed)

Menu: allow mouse hover to open submenu

Reported by: mtyson Owned by: bill
Priority: high Milestone: 1.9
Component: Dijit Version: 1.4.0
Keywords: menu submenu popup hover mouseover Cc:
Blocked By: Blocking:

Description (last modified by bill)

Add it as an option for hovering a menu item to open the sub menu, even if the menu/menubar isn't focused.

Attachments (1)

submenu-hover-diff.txt (1.1 KB) - added by mtyson 9 years ago.
Diff to add allowSubmenuHover prop to enable hover popup of submenus.

Download all attachments as: .zip

Change History (5)

Changed 9 years ago by mtyson

Attachment: submenu-hover-diff.txt added

Diff to add allowSubmenuHover prop to enable hover popup of submenus.

comment:1 Changed 9 years ago by bill

Milestone: 1.51.6

Are you talking about hovering over a MenuBar that isn't focused aka active? I tried pre-1.4, namely test_Menu.html in 1.3, and it doesn't open the submenu either: . And test_Menu.html in 1.2 doesn't even have a MenuBar.

comment:2 Changed 8 years ago by bill

Description: modified (diff)
Milestone: 1.6future
Summary: Allow Mouse Hover to Open SubMenuMenu: allow mouse hover to open submenu

Modified description since it doesn't seem like menus ever worked this way. Maybe back in the 0.4 days?

Note that hovering does open a submenu, but only after the menu itself has gotten focus by clicking it. This is the same way that menus on windows and mac work. But still, I guess it's an understandable enhancement request.

The patch is very small and looks good. Would need to add a test case though.

The main issue I see is with accessibility. Currently, opening a submenu puts focus on (the first item of) the submenu, which seems disruptive, especially if the user was focused somewhere else and was just (for example) sliding the mouse out of the way, and happened to move over a menu.

I think I'd like to have that "focus issue" fixed before adding this feature.

comment:3 Changed 6 years ago by bill

Milestone: future1.9
Owner: set to bill
Status: newassigned

comment:4 Changed 6 years ago by bill

Resolution: fixed
Status: assignedclosed

In [30900]:

Add passivePopupDelay property to allow menus to open merely by hovering them. Fixes #10979 !strict.

Note: See TracTickets for help on using tickets.