DataTable and popup accessibility

Description

From on June 16, 2015 1:10

I've started a first pass at making the table more accessible for screen readers etc and I'm stuck on introducing a row header as per the spec e.g. http://www.w3.org/WAI/tutorials/tables/two-headers/.

Is there a Wicket-way of adding an AttributeModifier to all <thead><tr><th> to introduce scope='col' and a modifying the student name cell to be a <th scope='row'>?

Thanks!

Copied from original issue: steveswinsburg/gradebookNGGIT2-109

Activity

Neal Caidin August 3, 2017 at 12:51 PM

This issue was imported from github issues. The use of Github issues as a bug tracker was an experiment with the GradebookNG (now just called Gradebook) tool development, released with Sakai 11. The information here may not be 100% accurate, please update or comment if any of the fields need updating.

Payten Giles April 28, 2016 at 5:00 PM

This is no longer an issue.

Steve Swinsburg January 28, 2016 at 3:00 PM

Discussed 29/1/2016
Probably doable, might be a little tricky. We are getting the accessibility review done so if they say that this is a priority we can address it.

Steve Swinsburg December 2, 2015 at 2:11 PM

OK I got this back. This means we need to rework the cells a bit which is not impossible, but is a big change. We may need to subclass Item and pass that in so we can override the onComponentTag method and set the tag name to 'th'. IMO lets leave this as an open issue and fix it after the main bits are done.

It's a little bit finicky, but it can be done:

In a future version we might want to change #newCellItem() to get a reference to the row's model - that would remove the need to lookup the row object from the grandparent.

Steve Swinsburg December 2, 2015 at 2:11 PM

From on June 22, 2015 22:23

Oo wowzers Thanks for chasing that down!

And yep no probs. I'll prepare the accessibility changes I've already done for a PR in the meantime.

Non-Issue

Details

Affects versions

Components

Assignee

Reporter

Created December 2, 2015 at 2:11 PM
Updated August 14, 2017 at 12:45 PM
Resolved August 4, 2017 at 8:15 PM