#14396 closed enhancement (fixed)
grid accessibility: need to be able to add a summary attribute to a DataGrid
Reported by: | Becky Gibson | Owned by: | Evan |
---|---|---|---|
Priority: | low | Milestone: | 1.8 |
Component: | Accessibility | Version: | 1.7.0 |
Keywords: | Cc: | mikeb | |
Blocked By: | Blocking: |
Description
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.
Attachments (1)
Change History (17)
comment:1 Changed 9 years ago by
Milestone: | 1.7.1 → 1.7.2 |
---|
comment:4 Changed 9 years ago by
Milestone: | 1.7.2 → 1.7.3 |
---|
1.7.2 RC released, bumping milestone on remaining tickets.
comment:5 Changed 9 years ago by
Milestone: | 1.7.3 → 1.8 |
---|---|
Type: | defect → enhancement |
This is an enhancement, so moving to a major release milestone.
comment:6 Changed 9 years ago by
Cc: | mikeb added; evan removed |
---|---|
Owner: | changed from Becky Gibson to Evan |
Status: | new → assigned |
comment:7 follow-up: 10 Changed 9 years ago by
Milestone: | 1.8 → 2.0 |
---|
1.8 is frozen. Move all enhancements to next release. If you need an exemption from the freeze for this ticket, contact me immediately.
comment:8 follow-up: 9 Changed 9 years ago by
dijit/_PaletteMixin.js has a summary too, you can probably do something similar:
// summary: String // Localized summary for the palette table summary: '', _setSummaryAttr: "paletteTableNode",
comment:9 follow-up: 11 Changed 9 years ago by
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).
comment:10 Changed 9 years ago by
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!
comment:11 Changed 9 years ago by
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.
comment:12 Changed 9 years ago by
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.
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).