Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#9176 closed defect (fixed)

[patch] [ccla] add aria-readonly property to the grid and to cells as appropriate

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

Description

Currently waiting for clarification from the ARIA spec on how this should be implemented. Proposal was made that for grid, the default readonly value should be true (rather than false as it is for input fields). Then, readonly=false would be set on editable grid cells rather than having to set readonly=true on non-editable cells.

Attachments (1)

9176_draft.patch (3.8 KB) - added by Becky Gibson 10 years ago.
contains console statements that will be removed before commit

Download all attachments as: .zip

Change History (5)

comment:1 Changed 10 years ago by Becky Gibson

Owner: changed from Bryan Forbes to Becky Gibson

May 19 version of the ARIA spec (http://www.w3.org/WAI/PF/aria/#grid) indicates that the default for grid is readonly=false, the grid IS editable, and the readonly value of the grid is propagated to the contained gridcells.

If the DataGrid? contains editable cells, there is no need to set the aria-readonly property on the grid since readonly=false is the default. Each non-editable cell must be marked with aria-readonly=true. The editable cells do NOT need to have an aria-readonly property added since they inherit from the grid.

If the grid does NOT contain any editable cells, set aria-readonly=true on the grid. Each individual cell will inherit this value and thus does not need the readonly property set.

Since the DataGrid? does not currently support editing column headers, those cells get aria-readonly=true, irregardless of the readonly setting of the grid.

Changed 10 years ago by Becky Gibson

Attachment: 9176_draft.patch added

contains console statements that will be removed before commit

comment:2 Changed 10 years ago by Becky Gibson

Summary: add aria-readonly property to the grid and to cells as appropriate[patch] [ccla] add aria-readonly property to the grid and to cells as appropriate

comment:3 Changed 10 years ago by Becky Gibson

Resolution: fixed
Status: newclosed

(In [18456]) Fixes #9176 Add ARIA readonly property on the grid and cells as appriopriate see ticket for implementation details. Fixes 9428 remove id on grid view pages as the ids were not used and duplicated when there were multiple views. !strict

comment:4 Changed 10 years ago by Adam Peller

Milestone: tbd1.4
Note: See TracTickets for help on using tickets.