-
Notifications
You must be signed in to change notification settings - Fork 132
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove redundant <dl> wrapper for the attribute table + prose block of each element #113
Comments
As I noted in #219, some chapters don't use this structure. I think I prefer the format that most elements in the paint server chapter use: a definition list, with the name of the attribute as the
If we want to keep the attribute name inside the blue boxes, then we shouldn't use a definition list format: a table doesn't make sense as a definition term. |
@AmeliaBR I suppose you mean a template like the one used for From accessibility point of view this should be fine though on a sequential reading one hears all headers first followed by the individual rows with values like attr name, value type and so on. IMO could be better eventhough a11y software does allow navigation in tables.
Note that the 2nd example is very common in CSS specs. It might even be the only pattern used by CSS specs and seems to be more accessible for sequential only reading. Would you disagree? |
If the nested If agreement is not reached fairly rapidly, this should be punted to the 2.1 milestone. |
A list with one entry is just confusing for screen reader users, especially when the "term" is a table containing
<dfn>
elements. The list structure is not visible at all for other users.The text was updated successfully, but these errors were encountered: