- Issue created by @rkoller
- πΊπΈUnited States katannshaw
I'm also seeing three instances of these focusable yet visually-hidden headings.
<h4 id="menu--shortcuts--2" class="toolbar-block__title visually-hidden focusable"></h4> <h4 id="menu--content" class="toolbar-block__title visually-hidden focusable"></h4> <h4 id="menu--admin" class="toolbar-block__title visually-hidden focusable"></h4>
Because these are meant to be visually-hidden headings, I'd recommend the following proposed solution:
1. Add missing heading text for each of them
2. Remove the CSS class.focusable
<h4 id="menu--shortcuts--2" class="toolbar-block__title visually-hidden">Shortcuts</h4> <h4 id="menu--content" class="toolbar-block__title visually-hidden">Content</h4> <h4 id="menu--admin" class="toolbar-block__title visually-hidden">Administration</h4>
- πͺπΈSpain ckrina Barcelona
Thanks @katannshaw! Updating the issue summary with the detailed recommended approach to make it easier to understand and address the issue.
- πΊπΈUnited States sjhunsinger Washington state
Hey there, I'm working on this issue at the Mentored Contribution during DrupalCon Portland 2024. I'll be working on this for the next few hours.
- πΊπΈUnited States scottatdrake
Working on this issue during the mentored contribution DrupalCon Portland 2024.
Hello. I'm working on this at the DrupalCon Mentor Contribution in Portland 2024.
I will be working on this for the next few hours.
- π―π΅Japan hktang
Hi, I am working on this at the mentor contribution session in Drupalcon in Portland 2024. I will be working on this for the next few hours.
- πΊπΈUnited States joshmiller Indianapolis, Indiana, USA
Hello! I'm working at the mentored contribution in Drupalcon Portland 2024. I will be working on this for the next few hours.
- π―π΅Japan hktang
hktang β changed the visibility of the branch 3439643-block-title-accessibility to hidden.
- π―π΅Japan hktang
hktang β changed the visibility of the branch 3439643-block-title-accessibility to hidden.
- Merge request !7973Draft: Resolve #3439643 "Block title accessibility" β (Open) created by scottatdrake
- πΊπΈUnited States mrcgomez
Hello! I'm working at the mentored contribution in Drupalcon Portland 2024. I will be working on this for the next few hours.
- πΊπΈUnited States emoleee Seattle, WA
Hello! I'm working at the mentored contribution in Drupalcon Portland 2024 too. I will be working on this for the next few hours.
- πΊπΈUnited States emoleee Seattle, WA
Hello! I'm working at the mentored contribution in Drupalcon Portland 2024. I will be working on this for the next few hours too.
- πΊπΈUnited States sjhunsinger Washington state
So far we have accomplished the following:
- We are spun up to do the work with DrupalPod/Tugboat.
- Everyone on our team has posted a comment.
Up next:
- We have begun to validate the issue so that we reproduce the issue properly. So far we have not been able to recreate. I used a screen reader extension to tab through the admin experience via Tugboat and was not able to recreate.
- We are going to continue our investigation after lunch and use the parent issue for clues to ensure we are recreating properly.
We'll be back at it after lunch!
- πΊπΈUnited States emoleee Seattle, WA
I am working on this issue at DrupalCon 2024 PDX and will be working with it for the next few hours as well.
- π―π΅Japan hktang
We've got some feedback and further information from @rkoller. We are summarizing these and provide an update soon.
- πΊπΈUnited States thehuffman Portland, OR
I am also taking a look at this issue as part of contribution day at DrupalCon Portland 2024.
So far I'm not seeing the
title
variable (see line 4) ever get populated in thenavigation--menu.html.twig
template. Either way, I'm also seeing a heading level is being skipped between the menu template and its parentblock--navigation.html.twig
template.We might consider not outputting a heading in the menu template, and instead provide an
aria-label
attribute for the<ul>
, which would use thetitle
var as its value. - πΊπΈUnited States scottatdrake
We are out of time today. We spoke with the original opener of this issue, rkoller, via slack. The thread can be found at: https://drupal.slack.com/archives/C7AB68LJV/p1715202081200079
rkoller is using voiceover on a mac, and looking at the headings using voice over's rotor feature. The uploaded video nicely illustrates this.
Some hopefully useful context:
https://www.w3.org/WAI/tutorials/menus/structure/#label-menus
This is an indication that the invisible
tag is desirable, but in the time I've had, I haven't found an explanation for specifically why this is useful or what the outcome should be. That section includes a link to https://www.w3.org/WAI/tutorials/page-structure/labels/ which might be more useful.
- π¬π§United Kingdom jacobupal Leeds
Removing novice tag as the next steps require some specialist expertise.