fix navigation for touch devices

contao-themes-net · Oct 9, 2020 · 22d2206 · 22d2206
1 parent 34bd9d4
commit 22d2206
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/src/Resources/public/js/theme.js b/src/Resources/public/js/theme.js
@@ -1,3 +1,8 @@
+function is_touch_device() {
+    return !!('ontouchstart' in window        // works on most browsers
+        || navigator.maxTouchPoints);       // works on IE10/11 and Surface
+};
+
 jQuery.noConflict();
 jQuery(document).ready(function ($) {
 
@@ -219,9 +224,7 @@ jQuery(document).ready(function ($) {
 
     $(".sidenav").sidenav();
 
-    var touch = matchMedia('(hover: none)').matches;
-
-    if(touch == false) {
+    if(is_touch_device() == false) {
         $("nav:not(.subnav) .desktop-menu a.dropdown-button").dropdown({
             inDuration: 300,
             outDuration: 225,
@@ -267,7 +270,7 @@ jQuery(document).ready(function ($) {
             closeOnClick: false
         });
 
-        $("a.dropdown-button").click( function() {
+        $("nav:not(.subnav)").on("touchstart","a.dropdown-button", function(e) {
             if( $(this).hasClass("open") ) {
                 window.location.href = $(this).attr('href');
             }

diff --git a/src/Resources/public/js/theme.min.js b/src/Resources/public/js/theme.min.js