Opened 9 years ago

Closed 6 years ago

Last modified 5 years ago

#11800 closed enhancement (fixed)

Menu: don't highlight/focus/select a default item in a right-click context menu

Reported by: Paul Lynch Owned by: bill
Priority: low Milestone: 1.9
Component: Dijit Version: 1.5
Keywords: Cc: Becky Gibson
Blocked By: Blocking:

Description

If you right click on a page in Firefox, you get a context menu where no item is highlighted. The arrow keys still work; hitting down arrow hightlights the first item. In Dojo, when a right-click context menu appears, the first item is highlighted, even if that item is disabled. It would be better if the default behavior were to not highlight any item, and to let the dojo user specify an item to be highlighted if they wish. At the very least, there needs to be an option for not highlighting a default item on the menu.

Change History (9)

comment:1 Changed 9 years ago by bill

Cc: Becky Gibson added
Component: GeneralDijit
Owner: anonymous deleted
Summary: Don't highlight/focus/select a default item in a right-click context menuMenu: don't highlight/focus/select a default item in a right-click context menu

Becky, what do you think? I tried shift-F10 in windows and it doesn't highlight an item either. (It works the same as right-click, where the first down arrow highlights the first item.) Does focus go to the menu itself?

comment:2 Changed 7 years ago by bill

Priority: highlow

comment:3 Changed 7 years ago by bill

Milestone: tbdfuture

comment:4 Changed 6 years ago by bill

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

I'll change it just for mouse, and have keyboard continue to mark the first item as selected.

comment:5 Changed 6 years ago by bill

Resolution: fixed
Status: assignedclosed

In [30901]:

Don't highlight a default item when a context menu is opened via the mouse. Continue to highlight the first item when it's opened by keyboard. Fixes #11800 !strict.

comment:6 Changed 6 years ago by bill

In [31087]:

allow for IE9+'s async focus, refs #11800

comment:7 Changed 6 years ago by bill

In [31106]:

allow for IE9+'s async focus in Form test, refs #11800

comment:8 Changed 6 years ago by bill

[31106] was actually for #16926, wrote the wrong number in the check in comment.

Last edited 6 years ago by bill (previous) (diff)

comment:9 Changed 5 years ago by Bill Keese <bill@…>

In 22e1395411ee81d1c6cd72bc08ef954591bb3563/dijit:

Error: Processor CommitTicketReference failed
Unsupported version control system "git": Can't find an appropriate component, maybe the corresponding plugin was not enabled? 
Note: See TracTickets for help on using tickets.