Skip to content

Match the Get Involved item by class so the mobile rule applies (develop) - #231

Merged
mlaetitia merged 1 commit into
developfrom
fix/t51eng-2081-hide-mobile-menu-item-class-develop
Sep 1, 2026
Merged

Match the Get Involved item by class so the mobile rule applies (develop)#231
mlaetitia merged 1 commit into
developfrom
fix/t51eng-2081-hide-mobile-menu-item-class-develop

Conversation

@mlaetitia

@mlaetitia mlaetitia commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Related to #

Proposed changes

Why are these changes being made?

#230 hid #menu-item-23441 below the nav breakpoint, but it had no effect on the item you can see.

primary_navigation and mobile_navigation render the same WP menu, so every item appears twice on the page.

Testing instructions

  • Below 1200px, open the mobile menu. Get Involved should not appear in the list. The pinned Get Involved button at the bottom should still be there.
  • Above 1200px, Get Involved should still appear in the primary menu, styled as a button by the Customizer CSS.

… the primary menu's copy and not the visible mobile one

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011rCHPCdbV5C5dDJjEmMDuy
@mlaetitia
mlaetitia merged commit 6c61cb0 into develop Sep 1, 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