-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Closed
Labels
[Feature] Site EditorRelated to the overarching Site Editor (formerly "full site editing")Related to the overarching Site Editor (formerly "full site editing")[Focus] Accessibility (a11y)Changes that impact accessibility and need corresponding review (e.g. markup changes).Changes that impact accessibility and need corresponding review (e.g. markup changes).[Status] In ProgressTracking issues with work in progressTracking issues with work in progress[Type] BugAn existing feature does not function as intendedAn existing feature does not function as intended
Description
The templates list page added in #36379 uses a <ul>
for the markup that displays the templates available to edit.
<table>
is the more semantically correct element to use here as we are presenting tabular data ("data with multiple dimensions") to the user. It also assists screen readers, as @talldan summarises in #36379 (comment):
When tabbing through a table, a screen reader user will be informed which column and row they're in, and when entering the table for the first time how many columns and rows there are. With a list, the user will only know how many list items, and won't be aware of columns.
Metadata
Metadata
Assignees
Labels
[Feature] Site EditorRelated to the overarching Site Editor (formerly "full site editing")Related to the overarching Site Editor (formerly "full site editing")[Focus] Accessibility (a11y)Changes that impact accessibility and need corresponding review (e.g. markup changes).Changes that impact accessibility and need corresponding review (e.g. markup changes).[Status] In ProgressTracking issues with work in progressTracking issues with work in progress[Type] BugAn existing feature does not function as intendedAn existing feature does not function as intended