Skip to content

a11y: tiny fixes from axe testing#5223

Draft
StephDriver wants to merge 2 commits intomasterfrom
b-5222_tiny_a11y_fixes
Draft

a11y: tiny fixes from axe testing#5223
StephDriver wants to merge 2 commits intomasterfrom
b-5222_tiny_a11y_fixes

Conversation

@StephDriver
Copy link
Member

closes #5222

@StephDriver
Copy link
Member Author

StephDriver commented Mar 9, 2026

list <ul> and <ol> must only directly contain <li>, <script> or <template> elements

Impact: serious
Docs: https://dequeuniversity.com/rules/axe/4.11/list?application=playwright
Failures (1):

Selector: .text-left > .list-inline
HTML: <ul class="list-inline">
Issue: Fix all of the following:
List element has direct children that are not allowed: p

There are 8 pages on clean which fail axe solely because of this, and with this update then pass.

@StephDriver StephDriver changed the title a11y: #5222 remove extraneous paragraphs from inside list a11y: tiny fixes from audit Mar 10, 2026
@StephDriver StephDriver changed the title a11y: tiny fixes from audit a11y: tiny fixes from axe testing Mar 10, 2026
@StephDriver
Copy link
Member Author

StephDriver commented Mar 10, 2026

Material Pagination

Lists were missing <li></li> tags around list contents (bad rebase?)

This affected the <div class="pagination-block> on four material templates, there were links to each page and <ul> tags but no <li> tags.

@StephDriver StephDriver force-pushed the b-5222_tiny_a11y_fixes branch from 5475a5f to db5ec6f Compare March 10, 2026 14:23
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.

Tiny A11y fixes during audit

1 participant