Opened 13 months ago
Last modified 12 months ago
#34832 closed Cleanup/optimization
Use `banner` landmark or `<header>` element for the admin header area — at Version 1
| Reported by: | Thibaud Colas | Owned by: | nobody |
|---|---|---|---|
| Component: | contrib.admin | Version: | 4.2 |
| Severity: | Normal | Keywords: | accessibility, screen reader, landmarks |
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | yes | UI/UX: | yes |
Description (last modified by )
Related: #34833. The Django admin uses a <div id="header"></div> for its header area. Switching this to a <header> tag or adding a role="banner" attribute to the div would help screen reader users navigate the interface – they would have an easier time bypassing the whole area when going through the page sequentially, and would also be able to jump straight to the header from any point on the page.
Usage of the correct HTML header tag is ideal, but might prove problematic if CSS was written with overly specific HTML elements. We can get the same benefits for screen reader users with role="banner".
Note:
See TracTickets
for help on using tickets.