- Issue created by @smustgrave
- πΊπΈUnited States smustgrave
Needs way more work and not entirely sure what a good BC way would be
- Status changed to Needs work
about 1 month ago 7:13am 20 June 2025 - π¬π§United Kingdom dahousecat
I needed to ensure that this was 100% accessible, and I was getting confused about the nested a in the li, so decided the only way to be sure it was correct was to just match the HTML from the example page here: https://www.w3.org/WAI/ARIA/apg/patterns/tabs/examples/tabs-automatic/
So the tablist should be a div, with each tab being a button.
This involved updating QuickTabs.php to adjust the html output, and also updating the JS to work with the updated html.
I've also added a setOptions method to the QuickTabsInstance as this allows for programmatically setting the label before render.
- πΊπΈUnited States tregonia
Tried the patch #6, and the merge request in #3, and neither patch applies to v4.0.1 on Drupal v11.1.7.
The goal of adding this patch was to hopefully resolve the issue:
Role with implied hidden content has keyboard focus
A fix for this would be huge for my department, this 'role="tab"' on the 'li' instead of the 'a' is the single biggest cause of lower accessibility scores in SiteImprove across every site that uses QuickTabs