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

improving accessibility of tables #4371

Closed
wants to merge 3 commits into from
Closed

Conversation

StephDriver
Copy link
Contributor

This will take some time!

Each table has a comment above it, so that it can be identified for work, and I can track which ones I've reviewed at any point.

right at the end of this, before merging, the comments will be removed.

closes #4055

@StephDriver StephDriver self-assigned this Aug 23, 2024
@StephDriver
Copy link
Contributor Author

Process:

  • find tables and add comment above each.
  • first review - expand comment to note obvious issues, and if none, which type of table (see below)
  • fix obvious issues, and update comment to type of table.
  • for each type of table, review in more detail, and fix any a11y markup required.
  • Review all as to whether captions and summaries are required (probably not for our use, but check).
  • remove the comments

Types of table (as per WCAG guidelines):

  • V1 - One header (either row or column)
  • V2 - header row and column row
  • V3 - Irregular headers
  • V4 - Multilevel headers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Table A11y updates
1 participant