Opened 8 months ago

Closed 8 months ago

#34833 closed Cleanup/optimization (fixed)

Use a `main` landmark in the admin interface

Reported by: Thibaud Colas Owned by: Sarah Abderemane
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: no

Description (last modified by Thibaud Colas)

Related: #34832. Currently the Django admin’s main content area is coded as <div class="main shifted" id="main"></div>. For screen reader users, it would be helpful to switch to a <main> tag or add role="main". They would be able to navigate straight to this area when going through the page by landmark.

Usage of the correct HTML main 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="main".

---

See ​recommendation from Axe on the importance of having a main landmark.

Change History (7)

comment:1 by Mariusz Felisiak, 8 months ago

Component: Uncategorized → contrib.admin
Triage Stage: Unreviewed → Accepted
Type: Uncategorized → Cleanup/optimization

comment:2 by Mariusz Felisiak, 8 months ago

Easy pickings: set

comment:3 by Thibaud Colas, 8 months ago

Description: modified (diff)
Keywords: accessibility screen reader landmarks added

comment:4 by Sarah Abderemane, 8 months ago

Owner: changed from nobody to Sarah Abderemane
Status: new → assigned

comment:5 by Sarah Abderemane, 8 months ago

Has patch: set

​PR

comment:6 by Mariusz Felisiak, 8 months ago

Triage Stage: Accepted → Ready for checkin

comment:7 by Mariusz Felisiak <felisiak.mariusz@…>, 8 months ago

Resolution: → fixed
Status: assigned → closed

In 91e617c7:

Fixed #34833 -- Made admin's main content render in <main> tag.

Note: See TracTickets for help on using tickets.
Back to Top