Opened 6 months ago

Last modified 4 months ago

#35507 assigned Bug

Use search element for admin's filter sidebar

Reported by: Tom Carrick Owned by: icastellanox
Component: contrib.admin Version: dev
Severity: Normal Keywords: accessibility
Cc: Tom Carrick, Paolo Melchiorre, Thibaud Colas Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

Using a search landmark instead of a div will improve the accessibility for screen readers.

We should ensure that it has a different aria-label than any other search landmarks or elements with the search role on the page (e.g. search bar) see: https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/Search_role#labeling_landmarks

I didn't check if there are any of these other than the search bar.

Change History (7)

comment:1 by Tom Carrick, 6 months ago

Owner: changed from nobody to Tom Carrick

comment:2 by icastellanox, 6 months ago

comment:3 by Paolo Melchiorre, 6 months ago

Cc: Paolo Melchiorre added
Has patch: set
Triage Stage: UnreviewedAccepted

comment:4 by Natalia Bidart, 6 months ago

For further information, see description of the search element in this link, specifically:

The <search> element semantically identifies the purpose of the element's contents as having search or filtering capabilities.

comment:5 by Sarah Boyce, 6 months ago

Cc: Thibaud Colas added
Patch needs improvement: set

This appears to revert #34835 which was introduced in 5.1, makes me think we should backport this to 5.1 before the beta freeze and mark this as a release blocker if we are instead saying this is in fact more accurate. We would also need to update the release note added in #34835.
Otherwise, we can close this ticket as wontfix.

comment:6 by Tom Carrick, 6 months ago

So... I feel uncomfortable with this phrasing a little. I think nav was an improvement, and this is a further improvement. It could be backported, I don't have a problem with that either, but I wouldn't go so far as to say that nav was wrong, just that this is "more right".

comment:7 by Sarah Boyce, 4 months ago

Owner: changed from Tom Carrick to icastellanox
Note: See TracTickets for help on using tickets.
Back to Top