Loading
fix: #3614658 Order the header menu to the design and drop the duplicate Events link
Fixes #3614658: https://www.drupal.org/project/horizonaid/issues/3614658
Header menu did not match the design:
- Events sorted before Countries. The design orders About, Resources, Programs, Countries, Events, Donate.
- "Events" appeared twice, and the call-to-action pill read "Events" instead of "Donate". The Events view (
views.view.events) shipped its own main-menu link at weight 0, duplicating the curatedmain--09--events.ymllink. The theme renders the last main-menu item as the CTA pill, so that stray link took it.
Changes:
- Swap the two shipped menu link weights so Countries (
-46) sorts before Events (-45). - Set
display.page_1.display_options.menu.type: noneonviews.view.eventsso the view stops shipping its own menu link. Donate then falls last on its own and becomes the CTA pill.
Verified in a browser on a Varbase 11 site: header renders About, Resources, Programs, Countries, Events, then Donate as the pill; Events appears once. Menu tree: Home(-50, disabled) → About(-49) → Resources(-48) → Programs(-47) → Countries(-46) → Events(-45) → Donate(0).
Note: the golden reference site still shows the old order and the duplicate link; this follows the design instead.
AI-Generated: Yes
Checkpoints:
- File an issue
- Addition/Change/Update/Fix
- Testing to ensure no regression
- Automated unit testing coverage
- Automated functional testing coverage
- UX/UI designer responsibilities
- Readability
- Accessibility
- Performance
- Security
- Developer Documentation
- User Guide Documentation
- Reviewed by human
- Code review by maintainers
- Full testing and approval
- Credit contributors
- Review with the product owner
- Release notes snippet
- Release
Edited by Rajab Natshah