Skip to content
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

Table headers not required when role="presentation" #93

Closed
robfentress opened this issue Feb 24, 2016 · 3 comments · Fixed by #586
Closed

Table headers not required when role="presentation" #93

robfentress opened this issue Feb 24, 2016 · 3 comments · Fixed by #586
Assignees
Labels
Blocked This issue is unable to be completed because of some external force. enhancement
Milestone

Comments

@robfentress
Copy link

UDOIT flags a table as being in error if it does not contain table headers (<th>). This rule should check whether or not role="presentation" has been used with the table and should only report it as an error if this role has not been set. With ARIA, it is now okay to use tables for layout, if the role is set to "presentation" (as long as the table makes sense when linearized), since this strips out any semantics.

screen shot 2016-02-24 at 11 09 44 am

@emilysachs emilysachs self-assigned this May 2, 2017
@bagofarms bagofarms added the Blocked This issue is unable to be completed because of some external force. label Mar 2, 2018
@bagofarms
Copy link
Member

This standard is in a rough draft state: w3c/aria-practices#182

When it is a published standard, we will merge Emily's code.

@bagofarms
Copy link
Member

I guess I never got back to this. That particular issue I mentioned in 2018 is still open, but it looks role="presentation" for tables made it into the standard: https://www.w3.org/TR/wai-aria-1.0/roles#presentation

I'll be reviewing Emily's code and seeing if it can still be merged after all these years.

@jasonmatos23-git jasonmatos23-git self-assigned this May 18, 2021
@jasonmatos23-git
Copy link
Contributor

Found an issue that was likely caused by resolving this issue. In some cases (see image) th are not detected.
Screen Shot 2021-06-10 at 2 36 12 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocked This issue is unable to be completed because of some external force. enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants