Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#8821 closed defect (fixed)

[patch] [ccla] focus not visible when first focus header cell of an empty grid

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

Description

load dojox/grid/tests/test_data_grid_empty.html. Tab to set focus to the columnheader - focus is not visible. However, if you use a tool such as MS Inspect which will outline the current focus item, the column header does have focus.

The problem is that when there is no data, focusManager.initFocusView does not get called. InitFocusView? calls _initColumnHeaders to set up the onfocus and onblur handlers for the column headers. Without those handlers in place, the focus /blur styles do not get set.

Attachments (1)

8821.patch (1.1 KB) - added by Becky Gibson 11 years ago.

Download all attachments as: .zip

Change History (4)

Changed 11 years ago by Becky Gibson

Attachment: 8821.patch added

comment:1 Changed 11 years ago by Becky Gibson

Summary: focus not visible when first focus header cell of an empty grid[patch] [ccla] focus not visible when first focus header cell of an empty grid

updated DataGrid?._onFetchComplete to call this.focus.initFocusView() when there is no data. Modified initFocusView so that this.focusView is modified if this.grid.views.getFirstScrollingView() returns undefined.

comment:2 Changed 11 years ago by Becky Gibson

Resolution: fixed
Status: newclosed

(In [16987]) fixes #8821 #7294 #8758 focus is now visible on column header cells of empty grid; tab and shift-tab work to navigate thru editable cells once in edit mode. !strict

comment:3 Changed 11 years ago by Adam Peller

Milestone: tbd1.3

batch move of tickets marked 'tbd' fixed in the 1.3 timeframe

Note: See TracTickets for help on using tickets.