Skip to content

Conversation

@Fran-A-Dev
Copy link
Contributor

  • Added a new handleMenuItemClick callback function that sets isMenuOpen to false
  • Added an onClick handler to each MenuItem component that calls handleMenuItemClick

Menu closes now when user clicks on item in mobile

@headless-platform-by-wp-engine

Check out the recent updates to your Headless Platform preview environment:

App Environment URL Build
faustjs.org preview-env-fix/primary-menu-close-on-mobile https://hx…wered.com ✅ (logs)

Learn more about preview environments in our documentation.

Copy link
Member

@colinmurphy colinmurphy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Fran-A-Dev LGTM 🚀

Tested preview link on Safari in iPhone 14 with IOS 18 and the issue was fixed 😄

@colinmurphy colinmurphy merged commit 2dc9d5f into main Feb 24, 2025
4 checks passed
@colinmurphy colinmurphy deleted the fix/primary-menu-close-on-mobile branch February 24, 2025 16:31
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.

3 participants