Opened 4 months ago

Last modified 4 weeks ago

#36367 assigned Cleanup/optimization

Improving the accessibility of the date_hierarchy layout in the admin.

Reported by: Antoliny Owned by: Chaitanya
Component: contrib.admin Version: 5.1
Severity: Normal Keywords: accessibility, screen reader, date_hierarchy
Cc: Eliana Rosselli, Thibaud Colas, Sarah Abderemane, Tom Carrick, Antoliny Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

When navigating to the date_hierarchy area in the admin page using a screen reader, no specific description is provided for that section. Currently, the only information announced is that it is a "navigation" area.


As a result, screen reader users may find it difficult to understand the purpose of the links contained within this area when they access it. The lack of context makes it unclear what role these links play.

Attachments (1)

date_hierarchy_screenreader_example.png (101.0 KB ) - added by Antoliny 4 months ago.

Download all attachments as: .zip

Change History (13)

comment:1 by Antoliny, 4 months ago

Owner: set to Antoliny
Status: newassigned

comment:2 by Sarah Boyce, 4 months ago

Cc: Eliana Rosselli Thibaud Colas Sarah Abderemane Tom Carrick added
Triage Stage: UnreviewedAccepted

comment:3 by Antoliny, 3 months ago

Cc: Antoliny added
Owner: Antoliny removed
Status: assignednew

comment:4 by Chaitanya, 5 weeks ago

Owner: set to Chaitanya
Status: newassigned

in admin/date_hierarchy.html

aria is one way of fixing it like below

<nav class="toplinks" aria-label="Date filter">

now the screen reader says "Date filter navigation" instead of "navigation".

or Else we can have a hidden <h> tag which also does the same thing but will be announced while going through headings of the page like below. (definitely not the best way)

<h3 class="visually-hidden">Date filter navigation</h3>

any other ways to fix??

Last edited 5 weeks ago by Chaitanya (previous) (diff)

in reply to:  4 ; comment:5 by Antoliny, 5 weeks ago

Replying to Chaitanya:

in admin/date_hierarchy.html

aria is one way of fixing it like below

<nav class="toplinks" aria-label="Date filter">

now the screen reader says "Date filter navigation" instead of "navigation".

or Else we can have a hidden <h> tag which also does the same thing but will be announced while going through headings of the page like below. (definitely not the best way)

<h3 class="visually-hidden">Date filter navigation</h3>

any other ways to fix??

Great! I think the changes above are necessary.

In addition, I believe the biggest issue here is how to convey the sorted order in the date_hierarchy.

Hmm... I've been thinking a lot about this recently as well, and I feel like we might need to slightly restructure this part.

in reply to:  5 ; comment:6 by Chaitanya, 5 weeks ago

Do we need to really convey the sorted order?? if yes then we can have the same aria label to say something like "sorted date filter navigation".
I don't think we need to announce the sorted order.

Last edited 5 weeks ago by Chaitanya (previous) (diff)

in reply to:  6 comment:7 by Antoliny, 5 weeks ago

Replying to Chaitanya:

Do we need to really convey the sorted order?? if yes then we can have the same aria label to say something like "sorted date filter navigation".
I don't think we need to announce the sorted order.

Yes, on second thought, since the date hierarchy already conveys its basic meaning, there's no need to provide additional information.
I think just adding a label should be enough. --> nav, aria-labelledby

However, I do think the word "hierarchy" should definitely be included, rather than simply calling it a date filter.
After all, users need to understand that this is ultimately a hierarchical structure.

Last edited 5 weeks ago by Antoliny (previous) (diff)

comment:9 by Sarah Boyce, 5 weeks ago

Patch needs improvement: set

comment:10 by Chaitanya, 4 weeks ago

Has patch: unset

comment:11 by Chaitanya, 4 weeks ago

Has patch: set
Patch needs improvement: unset

comment:12 by Sarah Boyce, 4 weeks ago

Needs tests: set
Patch needs improvement: set
Note: See TracTickets for help on using tickets.
Back to Top