Closed (won't fix)
Project:
Drupal core
Version:
7.x-dev
Component:
forum.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
6 Jul 2010 at 17:05 UTC
Updated:
29 Sep 2010 at 17:05 UTC
As per this description here http://www.webaim.org/techniques/tables/data.php
Forum accessibility would be improved by adding scope attributes.
I've added this to my sandbox and am contributing this patch http://drupal7.dev.openconcept.ca/forum/3
There are no visibility changes caused by this patch.
| Comment | File | Size | Author |
|---|---|---|---|
| add_scope2forum-1.patch | 1.94 KB | mgifford |
Comments
Comment #1
Everett Zufelt commented@mgifford
Can you please explain the current problem and how this patch improves the problem?
Comment #2
mgiffordI expect that scope is mostly useful when there is a
I'm not sure how this expresses itself in AT, but it's easy enough to find folks calling for the use of:
Veerle describes it here:
Then there's the Web Accessibility Tutoral's take on accessible table data and also going back to Section 508 (even though it is currently being reviewed). §1194.22 (g) Label Row and Column Headers:
There are articles like Use the scope attribute with table headers & others.
But really, this may not be either the best place to do it or might just be a nice idea that doesn't really provide direct benefits to users of AT.
Any additional thoughts on this?
Comment #3
Everett Zufelt commentedI would need to do some testing to see if AT actually respects scope, not that it is necessarily a bad idea to code to standard even if AT doesn't respect it.
For most tables I've seen in drupal core the only place I think it would be helpful is the corner cell.
Comment #4
mgiffordSo often the corner cell is left blank which is pretty useless. I've advocated for adding an invisible element here:
#821702: Replace Forum Icon Column with Topic Span
Please let us know what you find out about the use of scope in practice. And ya, good to code to a standard even if it isn't.
Comment #5
mgiffordBringing over comments from my sandbox generated by @jasonkiss - I'm bringing it over here because occasionally I have to restore the D7 sandbox and start from fresh.
In reference to this last item I added issue #851174: Update options checkbox doesn't include labels & #851164: Tableselect: "select all rows in this table" header checkbox causing accessibility problems
Comment #6
Everett Zufelt commentedI see no benefit of using the scope="col" attribute in any data table that isn't complex. th for the column heading cell is appropriate for 2d tables.