Role not removed from uls

Created on 17 October 2024, 6 months ago

Per the comment in this resolved issue https://www.drupal.org/project/tb_megamenu/issues/3192235#comment-13963908 → - aren't the list roles on menu uls supposed to be removed? They still exist, and because their parent list items don't have a role, accessibility tools are marking the mega menu as out of compliance per: https://www.w3.org/WAI/standards-guidelines/act/rules/bc4a75/proposed/

Check markup: role="list" still exists on the ul tags.

Remove $vars['attributes']['role'] = 'list'; from subnav, submenu, and nav functions.

🐛 Bug report

Status

Active

Version

3.0

Component

Code

Created by

🇺🇸United States toddwalden Kansas City, MO

Live updates comments and jobs are added and updated live.

Accessibility
It affects the ability of people with disabilities or special needs (such as blindness or color-blindness) to use Drupal.

Merge Requests

Issue created by @toddwalden
Comment 4 months ago →
sourabhsisodia_
Merge request !94tb_megamenu-3481553/3481553-role-not-removed: removed role from uls. → (Open) created by sourabhsisodia_
Comment 4 months ago →
sourabhsisodia_
Status changed to RTBC 3 months ago9:56am 21 January 2025
Comment 3 months ago →
🇮🇳India debrup
I have reviewed the changes. The fix is working as expected, due to the role attribute being successfully removed from the ul tags.

The test was performed on a Drupal 11.1.1 site.Attached some screenshots of the final markups.

Moving the issue to RTBC.

Production build 0.71.5 2024