#10483 closed defect (fixed)
[patch] [ccla] Grid: shift+F10 no longer invokes grid context menu (breaks keyboard a11y)
Reported by: | Becky Gibson | Owned by: | Becky Gibson |
---|---|---|---|
Priority: | high | Milestone: | 1.4.1 |
Component: | DojoX Grid | Version: | 1.4.0b |
Keywords: | a11y | Cc: | |
Blocked By: | Blocking: |
Description
open dojox/grid/tests/test_grid_column_display.html. Navigate to a grid header using the keyboard, with focus on a grid column header cell, press shift+F10 - a grid specific context menu should be displayed but the default browser context menu is displayed.
This is related to #9756 because focus is no longer on the actual grid column header cell. Need to modify code so grid listens for oncontext event rather than view.
Attachments (2)
Change History (5)
Changed 11 years ago by
Attachment: | 10483.patch added |
---|
comment:1 Changed 11 years ago by
Summary: | Grid: shift+F10 no longer invokes grid context menu (breaks keyboard a11y) → [patch] [ccla] Grid: shift+F10 no longer invokes grid context menu (breaks keyboard a11y) |
---|
Changed 11 years ago by
Attachment: | 10483_test.patch added |
---|
updated test files to check that shift+F10 opens header context menu
comment:2 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:3 Changed 11 years ago by
Note: See
TracTickets for help on using
tickets.
Attached a patch that modifies the node that the contextMenu is bound to when the grid header gets focus. If grid header gets keyboard focus, the contextMenu is bound to the grid.domNode because that controls the focus and keyevents for the header cells. When the header loses focus, the contextMenu is bound to the grid.viewsHeaderNode so clicking the right mouse button with the mouse over a header cell will invoke the context menu. Also did some cleanup to remove duplicate code when the header loses focus. During this fix I uncovered #10515 which existed before this fix and still exists.