Active
Project:
Toolbar
Version:
1.x-dev
Component:
Code
Priority:
Normal
Category:
Plan
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
29 Sep 2019 at 17:52 UTC
Updated:
21 Sep 2026 at 11:53 UTC
Jump to comment: Most recent
Toolbar module has a number of accessibility issues. Some of these may be addressed individually in child issues, but it will be a good idea to consider them holistically.
<img>/ <svg>, or the CSS content property.
However some buttons are badly broken, because visually they are just an icon. When the icon is missing users can't perceive or understand the button:
<a href="" role="button"> which only responds to the return key. It needs an extra keyboard handler. Spotted in #3066954: Admin toolbar not usable with latest versions of JAWS due to mis-use of aria-owns.
Comments
Comment #2
andrewmacpherson commentedStarting a meta-issue to improve toolbar accessibility. Some of these can be fixed separately but it's good to have a parent issue for them. Hopefully that will help to avoid conflating issues.
There are a few related issues already, so I'll gather those up.
Comment #3
andrewmacpherson commentedComment #4
andrewmacpherson commentedComment #5
andrewmacpherson commentedAdding the failure of WCAG Focus Order. Historically, I think the DOM order was chosen to prioritize some actions for blind users with a screen reader. The rationale was that users are more likely to want the edit button that the user profile links. However it makes a confusing tabbing order visually.
I'm not convinced that it's more important to reach the edit button before the logout button. The weird visual tabbing order is more important to fix IMO. However we could consider putting the user account menu on the right hand side; that's a common design pattern nowadays, and it would mean we can make the visual order match the current DOM order.
Comment #6
andrewmacpherson commentedTwo interesting issues.
#3066954: Admin toolbar not usable with latest versions of JAWS due to mis-use of aria-owns - top-level buttons with a menu tray have crappy ARIA states and don't work with spacebar. The
aria-pressed="false"state never changes!#3046089: Accessibility of Main toolbar items and internal editing - a mixture of issues, which need a few spin-offs.
Comment #7
andrewmacpherson commentedRelated: the problem with toolbar buttons never changing
aria-pressed="false"has also been noticed at #3012027: Decouple tour triggering from the toolbar.Comment #8
andrewmacpherson commentedComment #9
andrewmacpherson commentedComment #10
andrewmacpherson commentedComment #11
andrewmacpherson commentedComment #12
andrewmacpherson commentedComment #13
andrewmacpherson commentedThe back-to-site behaves OK. It's just a link, without any ARIA role or properties.
Comment #14
andrewmacpherson commentedComment #15
andrewmacpherson commentedComment #16
andrewmacpherson commentedComment #18
andrewmacpherson commentedComment #19
andrewmacpherson commentedComment #20
andrewmacpherson commentedComment #21
andrewmacpherson commentedFiled some new child issues. Rearranging the known issues in the summary here.
Comment #22
andrewmacpherson commentedComment #24
bnjmnmComment #28
mherchelCreated and linked child issue #3269420: Toolbar icons may not meet contrast when in forced colors mode
Comment #32
mgiffordAdding reference to Windows High Contrast Mode.
Comment #33
kentr commentedThese don't have their own issues yet, correct?
#3 is a quick fix.
From looking at the template, removing the nested navs as suggested for #1 might also fix #2.
It's also throwing this Axe error, which resolves when I change the
divtonavand remove therole="group".Comment #34
mgifford@kentr definitely useful to set up sub issues to see that there is some progress on this issue that Andrew set up initially in 2019.
Comment #35
kentr commentedAdded child issue #3509700: Accessibility of landmark regions in toolbar.
Comment #37
quietone commentedThe Toolbar Module was approved for removal in #3476882: [Policy] Move Toolbar module to contrib.
This is Postponed. The status is set according to two policies. The Remove a core extension and move it to a contributed project and the Extensions approved for removal policies.
The deprecation work is in #3484850: [meta] Tasks to deprecate Toolbar module and the removal work in #3488828: [meta] Tasks to remove Toolbar module.
Toolbar will be moved to a contributed project before Drupal 12.0.0 is released.
Comment #38
quietone commentedToolbar has moved to contrib