From 181bfc16a5e0aa30e69490959d9d65b40ed06667 Mon Sep 17 00:00:00 2001
From: benfc1993 <41189326+benfc1993@users.noreply.github.com>
Date: Wed, 15 Sep 2021 08:46:55 +0100
Subject: [PATCH] fix(menu): change toggle to target closest action

This allows for mobile menu toggle elements to contain any number of nested elements and keep the existing behaviour
---
 assets/js/theme/global/mobile-menu-toggle.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/assets/js/theme/global/mobile-menu-toggle.js b/assets/js/theme/global/mobile-menu-toggle.js
index df1c03be26..f02f18d43d 100644
--- a/assets/js/theme/global/mobile-menu-toggle.js
+++ b/assets/js/theme/global/mobile-menu-toggle.js
@@ -146,7 +146,7 @@ export class MobileMenuToggle {
             this.$navList.removeClass('subMenu-is-open');
         }
 
-        if ($(event.target).hasClass('is-open')) {
+        if ($closestAction.hasClass('is-open')) {
             $parentSiblings.addClass('is-hidden');
             $parentAction.addClass('is-hidden');
         } else {