Opened 2 years ago

Closed 2 years ago

#34042 closed Bug (fixed)

Screen reader issues with admin navigation sidebar

Reported by: Thibaud Colas Owned by: Gloib
Component: contrib.admin Version: 4.0
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: no UI/UX: yes

Description

There are three issues with the admin’s toggle-able sidebar nav for screen reader users:

  • Its toggle button doesn’t set aria-expanded to indicate its state. This is important so people can tell whether the current state is "opened" or "closed".
  • The sidebar is wrapped in a <nav> landmark (good!) but that landmark should ideally have a label set with aria-label="{% translate 'Sidebar' %}" so users can more easily decide whether this is the part of the UI they want to go to.
  • The sidebar shouldn’t be navigable when closed (to replicate the experience of sighted users). This is because we’re preventing navigation by setting tabindex="-1" on the sidebar interactive elements. Instead, we should set the whole region to display: none or visibility: hidden when closed.

Change History (5)

comment:1 by Mariusz Felisiak, 2 years ago

Triage Stage: UnreviewedAccepted

comment:2 by Gloib, 2 years ago

Has patch: set
Owner: changed from nobody to Gloib
Status: newassigned

comment:3 by Mariusz Felisiak, 2 years ago

Patch needs improvement: set

comment:4 by Mariusz Felisiak, 2 years ago

Patch needs improvement: unset
Triage Stage: AcceptedReady for checkin

comment:5 by Mariusz Felisiak <felisiak.mariusz@…>, 2 years ago

Resolution: fixed
Status: assignedclosed

In c4aac2ac:

Fixed #34042 -- Improved accessibility of admin's navigation sidebar.

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