From b76041907db48683b969efd8dc603a65d9468e08 Mon Sep 17 00:00:00 2001
From: thex <andreas.kar@gmx.at>
Date: Tue, 10 Sep 2019 18:35:03 +0200
Subject: [PATCH] disable menu on mobile devices

---
 engines/common/nucleus/particles/menu.html.twig | 2 +-
 engines/wordpress/nucleus/particles/menu.yaml   | 6 ++++++
 themes/hydrogen/common/scss/hydrogen/_nav.scss  | 7 +++++++
 3 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/engines/common/nucleus/particles/menu.html.twig b/engines/common/nucleus/particles/menu.html.twig
index d5001968a..4047be4c6 100644
--- a/engines/common/nucleus/particles/menu.html.twig
+++ b/engines/common/nucleus/particles/menu.html.twig
@@ -123,7 +123,7 @@
 {% import _self as macro %}
 
 {% if menu.root.count() %}
-<nav class="g-main-nav" role="navigation"{{ particle.mobileTarget ? ' data-g-mobile-target' : '' }} data-g-hover-expand="{{ particle.hoverExpand|default('true') ? 'true': 'false' }}">
+<nav class="g-main-nav{{ particle.disableMobile ? ' g-hide-mobile' : '' }}" role="navigation"{{ particle.mobileTarget ? ' data-g-mobile-target' : '' }} data-g-hover-expand="{{ particle.hoverExpand|default('true') ? 'true': 'false' }}">
     <ul class="g-toplevel">
         {{ macro.displayItems(menu.root, menu, _context) }}
     </ul>
diff --git a/engines/wordpress/nucleus/particles/menu.yaml b/engines/wordpress/nucleus/particles/menu.yaml
index 5c8f438d2..46e9f3414 100644
--- a/engines/wordpress/nucleus/particles/menu.yaml
+++ b/engines/wordpress/nucleus/particles/menu.yaml
@@ -70,3 +70,9 @@ form:
       label: Force Target Attribute
       description: 'Adds ''target="_self"'' attribute to all menu items. Fixes an issue with pinned tabs in Firefox where external links always open in a new tab.'
       default: 0
+
+    disableMobile:
+      type: input.checkbox
+      label: Disable on Mobile
+      description: Check this field to completely disable the menu on mobile devices. This prevents menu flickering but you might end with no menu when JS is not supported by the respective device. 
+      default: 0
\ No newline at end of file
diff --git a/themes/hydrogen/common/scss/hydrogen/_nav.scss b/themes/hydrogen/common/scss/hydrogen/_nav.scss
index b74d9229f..56aa03fa4 100644
--- a/themes/hydrogen/common/scss/hydrogen/_nav.scss
+++ b/themes/hydrogen/common/scss/hydrogen/_nav.scss
@@ -93,6 +93,13 @@
 	}
 }
 
+.g-main-nav.g-hide-mobile{
+    @include breakpoint(mobile-only) {
+        display: none;
+        visibility: hidden;
+    }
+}
+
 #g-navigation, #g-header {
 	.align-left {
 		.g-toplevel {