Skip to content

a11y: Fix accessibility violations found by the playwright axe page scan - #1597

Closed
coseeian wants to merge 4 commits into
processing:mainfrom
coseeian:playwright-scan-fix
Closed

coseeian wants to merge 4 commits into
processing:mainfrom
coseeian:playwright-scan-fix

Conversation

@coseeian

@coseeian coseeian commented Sep 14, 2026 •

Copy link
Copy Markdown
Collaborator

This PR clears up the heading-order, and link-name, landmark-unique accessibility violations from a11y page tests:

  • Fixed callout heading order: Callout titles were hardcoded as <h5>, breaking document outlines. Converted them to <p class="callout-title"> with role="note" on the container so visual styling stays identical while removing them from the heading structure.
  • Fixed empty reference links: Items without a submodule were generating empty focusable links (/reference/#), creating invisible tab stops and triggering link-name failures. We now only render these links when a submodule actually exists.
  • Excluded iframe sketch previews from axe scans: srcdoc preview frames generated by p5 were causing duplicate landmark warnings (<main> and canvas regions). Excluded frame contents from the scan scope while keeping iframe elements in scope.

@coseeian coseeian changed the title fix: heading-order a11y violations on tutorial pages fix: Fix accessibility violations found by the playwright axe page scan Sep 14, 2026
@coseeian coseeian changed the title fix: Fix accessibility violations found by the playwright axe page scan a11y: Fix accessibility violations found by the playwright axe page scan Sep 14, 2026
@coseeian coseeian added the Accessibility: Best Practice No obvious current impact, but which may lead to problems in the future label Sep 14, 2026
@coseeian

Copy link
Copy Markdown
Collaborator Author

I'm going to break this up into 3 separate PRs to keep the reviews lightweight and manageable.

@coseeian coseeian closed this Sep 14, 2026
@coseeian
coseeian deleted the playwright-scan-fix branch September 14, 2026 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Accessibility: Best Practice No obvious current impact, but which may lead to problems in the future

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant