Opened 8 months ago
Last modified 6 months ago
#35507 assigned Bug
Use search element for admin's filter sidebar
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.
According to the ticket's flags, the next step(s) to move this issue forward are:
- To improve the patch as described in the pull request review comments or on this ticket, then uncheck "Patch needs improvement".
If creating a new pull request, include a link to the pull request in the ticket comment when making that update. The usual format is:
[https://github.com/django/django/pull/#### PR]
.
Change History (7)
comment:1 by , 8 months ago
Owner: | changed from | to
---|
comment:2 by , 8 months ago
comment:3 by , 8 months ago
Cc: | added |
---|---|
Has patch: | set |
Triage Stage: | Unreviewed → Accepted |
comment:4 by , 8 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 , 8 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 , 8 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 , 6 months ago
Owner: | changed from | to
---|
I've made the pull request: https://github.com/django/django/pull/18256