Replace megamenu accordion heading tag with class #830

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

CySSoO merged 1 commit into master from cyssoo/remove-title-levels-in-menu-tpl

Jan 16, 2026

Contributor

CySSoO commented Jan 16, 2026

Motivation

Remove semantic heading usage inside the mobile megamenu accordion and rely on CSS classes (e.g. .h2) for visual styling to avoid introducing heading levels in menu templates.

Description

In views/templates/hook/prettyblocks/prettyblock_megamenu_item.tpl replaced the <h2 class="accordion-header" ...> wrapper with a <div class="accordion-header h2" ...> to keep Bootstrap accordion behavior while using a non-heading element for styling.

Testing

No automated tests were executed.


          Replace megamenu heading tag with class

e7cfaf4

CySSoO added the codex label

— with

ChatGPT Codex Connector

CySSoO merged commit 252d796 into master

2 of 6 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels