Opened 10 years ago

Closed 3 years ago

#8464 closed defect (patchwelcome)

MenuItem: shouldn't be able to invoke context menu when focus is on a menu item.

Reported by: Joseph Scheuhammer Owned by: bill
Priority: high Milestone: 1.13
Component: Accessibility Version: 1.2.3
Keywords: context menu, a11y, keyboard Cc: Becky Gibson, bill, davidb
Blocked By: Blocking:

Description

A context menu can be invoked using the keyboard: control+space on the Mac (FF), or shift+f10 on Windows (FF and IE). If a context menu is posted and the keystroke command is pressed again, the original context menu is dismissed and replaced with another context menu. The new context menu is displayed slightly to the right and down.

If the context menu keyboard command is repeatedly pressed, the context menu appears to slowly march down and to the right.

The reason this occurs: the first menu item in the menu has focus when the keystroke is made, and the browser emits an 'oncontextmenu' event. Either the menu item itself responds and posts the new context menu, or it passes the event on to some widget that does (I'm not sure which).

I don't think menuitems, menus nor menubars should have context menus associated with them. 'oncontextmenu' events should be "absorbed" when the target is a menubar, menu, or menuitem, and any current context menu should stay posted.

Steps to reproduce:

  1. Load the ".../dijit/tests/test_Menu.html" test file.
  2. place focus on the text entry field containing the text "top-left".
  3. press control+space (FF/Mac) or shift+f10 (WinXP/FF/IE) to invoke the context menu.
  4. press control+space or shift+f10 again.
  5. repeat last step as desired.

Result: the context menu is repeatedly dismissed and posted, and moves slightly over and down.

Change History (4)

comment:1 Changed 10 years ago by Becky Gibson

Milestone: tbd1.4

comment:2 Changed 10 years ago by Becky Gibson

Milestone: 1.4future

comment:3 Changed 8 years ago by Chris Mitchell

Owner: changed from Becky Gibson to bill

reassign due to inactive committer. please triage accordingly.

comment:4 Changed 3 years ago by dylan

Milestone: future1.12
Resolution: patchwelcome
Status: newclosed

A pull request would be welcome for this if it is still an issue, but given the lack of activity over the past 5+ years, I'm closing this one.

Note: See TracTickets for help on using tickets.