Opened 11 years ago

Closed 10 years ago

#7924 closed defect (fixed)

[patch] [ccla] shift-tab through editable cells should not include column header

Reported by: Becky Gibson Owned by: Bryan Forbes
Priority: high Milestone: 1.3
Component: DojoX Grid Version: 1.2.0
Keywords: a11y Cc: Becky Gibson
Blocked By: Blocking:

Description

in the dojox/grid/tests/test_edit.html when you press enter on a cell to invoke edit mode, subsequent tab presses will move you from cell to cell still in edit mode. This is correct and matches the DHTML style guide suggested behavior (http://dev.aol.com/dhtml_style_guide#grid). However, when you press shift-tab, focus goes from the current cell to the header cell of the current column and then to the previous cell. It should not go to the header cell before going to the previous cell when pressing shift-tab.

Attachments (1)

7924.patch (709 bytes) - added by Becky Gibson 10 years ago.

Download all attachments as: .zip

Change History (4)

comment:1 Changed 11 years ago by Bryan Forbes

Milestone: tbdfuture

Changed 10 years ago by Becky Gibson

Attachment: 7924.patch added

comment:2 Changed 10 years ago by Becky Gibson

Milestone: future1.3
Priority: normalhigh
Summary: shift-tab through editable cells should not include column header[patch] [ccla] shift-tab through editable cells should not include column header

Just need to check for editing in _FocusManager.previousKey() before checking this.isNavHeader() to prevent setting focus to headers when editing. Marked as 1.3, high so Bryan would take a look - reassign as desired. There are other issues related to editing and keyboard navigation - see #8758 and #8759

comment:3 Changed 10 years ago by Becky Gibson

Resolution: fixed
Status: newclosed

fixed with [16987]

Note: See TracTickets for help on using tickets.