Opened 20 months ago
Closed 20 months ago
#34033 closed Bug (fixed)
Admin light/dark theme switch needs to convey its state for screen reader users
Reported by: | Thibaud Colas | Owned by: | Florian Perucki |
---|---|---|---|
Component: | contrib.admin | Version: | dev |
Severity: | Release blocker | Keywords: | accessibility, screen reader, translations |
Cc: | Sarah Abderemane | 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 (last modified by )
Follow-up to PR #15843 / ticket #33690 – there are two issues with the theme switcher for screen reader users:
- The label isn’t translated
- The label doesn’t convey the button’s current state (light, dark, auto)
- (potential issue – the SVG elements should be hidden from screen reader users if they don’t have alternative text)
This is important for partly-sighted screen reader users, who will expect the visual appearance of the button and its screen reader announcement to both convey the current state. Particularly between the user’s "auto" system theme and a forced theme.
Here is the current accessibility tree representation (Google Chrome) as a reference:
button "Toggle Light / Dark / Auto color theme" StaticText "Toggle Light / Dark / Auto color theme" img "" Ignored
Change History (7)
comment:1 by , 20 months ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:2 by , 20 months ago
Description: | modified (diff) |
---|
comment:3 by , 20 months ago
Has patch: | set |
---|
comment:4 by , 20 months ago
comment:5 by , 20 months ago
Cc: | added |
---|---|
Severity: | Normal → Release blocker |
Triage Stage: | Unreviewed → Accepted |
Thanks for the report. Bug in bc7aa2a5e91cf65fc7510edaf1776528c7ad07b4.
comment:6 by , 20 months ago
Triage Stage: | Accepted → Ready for checkin |
---|
PR: https://github.com/django/django/pull/16085