Skip to content

Make the mobile menu CTA editable from the menu settings (develop) - #226

Merged
mlaetitia merged 4 commits into
developfrom
add/t51eng-2081-menu-cta-develop
Aug 13, 2026
Merged

Make the mobile menu CTA editable from the menu settings (develop)#226
mlaetitia merged 4 commits into
developfrom
add/t51eng-2081-menu-cta-develop

Conversation

@mlaetitia

@mlaetitia mlaetitia commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Related to T51ENG-2081

Proposed changes

  • Make the mobile menu's CTA button editable via a "Mobile menu CTA" link field on the primary menu's settings; no field value means no button

Why are these changes being made?

  • The CTA label/URL was hardcoded in the templates; editors should control it (including removing it) without a deploy

Testing instructions

  • In Appearance → Menus (classic screen), set the "Mobile menu CTA" link on the primary menu and verify the mobile menu's bottom button uses it (including new-tab if checked)
  • Clear the field and verify the button disappears
  • Note: after deploy the button is hidden until the field is set on the menu

mlaetitia and others added 4 commits August 13, 2026 17:46
…u's settings, keeping Get Involved as the fallback

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…ack to a hardcoded link

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@mlaetitia
mlaetitia merged commit 88c1551 into develop Aug 13, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant