Skip to content

Give explainer table header cells scope="col" - #1183

Merged
yakew7 merged 1 commit into
yakew7:mainfrom
minutechreview:fix/explainer-th-scope-1172
Oct 25, 2026
Merged

yakew7 merged 1 commit into
yakew7:mainfrom
minutechreview:fix/explainer-th-scope-1172

Conversation

@minutechreview

Copy link
Copy Markdown
Contributor

Summary

Explainer table header cells now carry scope="col", so screen readers tie each cell to its column header.

Type

  • Audit
  • Explainer
  • Bug fix
  • Other

Details

  • scripts/build_explainers.py: the <th> cells in a rendered markdown table's <thead> get scope="col".
  • assets/explainers-ui.js: the same change in the client-side renderMarkdown port, to keep the two renderers in parity.
  • Regenerated the explainer pages with scripts/build_explainers.py (50 pages contain tables). The only diff in the generated HTML is the added attribute. A rebuild also bumped the decision-curve-analysis-by-group lastmod/dateModified because of today's commit to its markdown; I left that out to keep this change to the one fix.
  • New test test_render_markdown_gives_every_table_header_a_column_scope renders a fixture table and checks both renderers. It fails without the change.

Checks: tests/test_build_explainers.py 26 passed, ruff check, check_em_dash.py and check_generated_files_current.py pass. grep -l '<table' explainers/*.html | xargs grep -L 'scope=' | wc -l now prints 0.

Linked issue

Closes #1172

Signed-off-by: minutechreview <141821420+minutechreview@users.noreply.github.com>
@vercel

vercel Bot commented Oct 25, 2026

Copy link
Copy Markdown
Contributor

@minutechreview is attempting to deploy a commit to the yashkewlani2020-gmailcom's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown

@yakew7 @Shreyash0712 @ahmdkaml @propcgamer20-png - new PR touching a path you own, please review.

@yakew7
yakew7 merged commit 337d6f3 into yakew7:main Oct 25, 2026
19 of 20 checks passed
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.

Explainer tables have header cells without a scope

2 participants