Opened 5 hours ago

Last modified 4 hours ago

#36630 assigned Bug

Semantic structure and accessibility improvements for the admin navigation sidebar. — at Version 2

Reported by: Antoliny Owned by: Antoliny
Component: contrib.admin Version: 5.2
Severity: Normal Keywords: accessibility
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Antoliny)

Currently, the admin navigation sidebar is built using <table> tags.
I don’t think this structure is appropriate from an accessibility perspective.
It has the following issues:

  • When accessing the table (app) element, it is described as having three columns, but I’m not sure exactly what columns exist in the current structure.

I feel that it was designed with a flawed structure from the start, providing an inaccurate description.
(For reference, the current columns are "model name", "add link" and "change or view list".
On the admin index page, this description is somewhat accurate, but in the sidebar, the change or view list link does not exist, so I believe it is definitely an incorrect description.)


  • When accessing a row element, the table structure provides a description of which column each item is in. However, since the sidebar does not have the last column, users expect three items in a row, but in reality, only two items are accessible.


  • When navigating elements with a screen reader, they are traversed in the wrong order.

Currently, when using the standard element navigation of a screen reader, it moves to the end of the table without accessing the internal elements.
I suspect this issue might be caused by the <a> tag being placed directly under the table.

  • Incorrect navigation order during tab navigation.

When using tab navigation, we would expect to navigate from the app to the models under it, but in reality, the models are navigated first, and the apps for those models are navigated last.

To resolve this issue, I believe the structure of app_list(sidebar, index, app index), should be changed to <h>, <ul>, <li>.

Change History (4)

by Antoliny, 5 hours ago

Attachment: table_wrong_description.png added

by Antoliny, 5 hours ago

comment:1 by Antoliny, 5 hours ago

Owner: set to Antoliny
Status: newassigned

comment:2 by Antoliny, 5 hours ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.
Back to Top