Opened 5 months ago
Last modified 3 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 , 5 months ago
Owner: | changed from | to
---|
comment:2 by , 5 months ago
comment:3 by , 5 months ago
Cc: | added |
---|---|
Has patch: | set |
Triage Stage: | Unreviewed → Accepted |
comment:4 by , 5 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 , 5 months ago
Cc: | 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 , 5 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 , 3 months ago
Owner: | changed from | to
---|
I've made the pull request: https://github.com/django/django/pull/18256