grid accessibility: need to be able to add a summary attribute to a DataGrid

[Becky Gibson]

The user should be able to add a summary attribute onto a DataGrid? in order to meet the WGAG 2.0 1.3.1 success criterion (​http://www.w3.org/TR/2008/REC-WCAG20-20081211/#content-structure-separation-programmatic).

Currently if you add a summary attribute into the markup for a DataGrid?, the attribute is not part of the generated grid.

The enhanced datagrid needs this capability as well.

[bill]

These didn't make it into the 1.7.1RC, so bumping them to 1.7.2 (as stated in the email I sent yesterday).

[Colin Snover]

Bulk update of open ticket priorities.

[Colin Snover]

Reducing priority.

[Colin Snover]

1.7.2 RC released, bumping milestone on remaining tickets.

[Colin Snover]

This is an enhancement, so moving to a major release milestone.

[Colin Snover]

1.8 is frozen. Move all enhancements to next release. If you need an exemption from the freeze for this ticket, contact me immediately.

[bill]

dijit/_PaletteMixin.js has a summary too, you can probably do something similar:

// summary: String
//              Localized summary for the palette table
summary: '',
_setSummaryAttr: "paletteTableNode",

[Evan]

Replying to bill:

dijit/_PaletteMixin.js has a summary too, you can probably do something similar:

// summary: String
//              Localized summary for the palette table
summary: '',
_setSummaryAttr: "paletteTableNode",

Thanks Bill,

As Grid is not using _CSSStateMixin at this moment, that might involve certain changes. So I just go ahead with the most simple way(with the patch).

[Evan]

Replying to csnover:

1.8 is frozen. Move all enhancements to next release. If you need an exemption from the freeze for this ticket, contact me immediately.

Hi Colin,

I think this is more like a defect to add a missed ARIA attr, the ​fix is trivial. Please let me know if it's OK to go ahead getting this into 1.8?

Thanks!

[Evan]

Proposed fix patch

[Evan]

Replying to Evan:

Thanks Bill,

As Grid is not using _CSSStateMixin at this moment, that might involve certain changes. So I just go ahead with the most simple way(with the patch).

Sorry, please just ignore this, nothing related with _CSSStateMixin, it's working nicely with the updated patch.

[Colin Snover]

I don’t have a problem with this being landed except that I would r- the patch that was uploaded for providing a bad default value on the summary and for seemingly not actually providing updated unit tests to check that setting this property actually works.

[Evan]

In [28908]:

Fixes #14396 !strict, summary attr working well with to all grid types

[Evan]

Also updated in the ​grid doc

[Evan]

In [29626]:

Refs #14396 also adding role="presentation" to FilterBar? template, backported to 1.8.x, 1.7.x branch

[Evan]

In [29746]:

Refs #14396 !strict, using focusNode so that JAWS can read the filter input field