Merge pull request #2511 in SW/shopware from bugfix/next/sw-11181-advanced-menu-fix to next

* commit '19cf7e161f704ef810a41ccfd91496de0f88344b':
  SW-11181 - fixed advanced menu on desktop, tablets and surface

Author: Marcel Schmäing

engine/Shopware/Plugins/Default/Frontend/AdvancedMenu/Views/frontend/_public/src/js/jquery.advanced-menu.js

@@ -60,14 +60,7 @@
              *
              * @type {String}
              */
-            'itemHoverClass': 'is--hovered',
-
-            /**
-             * Tolerance the touch should be counted as a tap.
-             *
-             * @type {Number}
-             */
-            'touchTimeTolerance': 100
+            'itemHoverClass': 'is--hovered'
         },
 
         /**
@@ -113,14 +106,6 @@
              */
             me._$closeButton = me.$el.find(me.opts.closeButtonSelector);
 
-            /**
-             * Timestamp of the last touch start on a list item.
-             *
-             * @private
-             * @property _touchStartTimestamp
-             * @type {Number}
-             */
-            me._touchStartTimestamp = 0;
 
             /**
              * PreviousTarget will be used for pointer events to prevent
@@ -151,13 +136,12 @@
             $.each(me._$listItems, function (i, el) {
                 $el = $(el);
 
-                me._on($el, 'touchstart', $.proxy(me.onTouchStart, me));
-
                 me._on($el, 'mouseenter', $.proxy(me.onListItemEnter, me, i, $el));
 
                 if (window.navigator.pointerEnabled || window.navigator.msPointerEnabled) {
-
                     me._on($el, 'click', $.proxy(me.onClickNavigationLink, me, i, $el))
+                } else {
+                    me._on($el, 'click', $.proxy(me.onClick, me, i, $el));
                 }
             });
 
@@ -167,14 +151,26 @@
         },
 
         /**
-         * Called when a touch started on a list item.
-         * Updates the timestamp property to determine a tap on touch end.
+         * Called when a click event is triggered.
+         * If touch is available preventing default behaviour.
          *
-         * @public
-         * @method onTouchStart
+         * @param event
+         */
+        onClick: function () {
+            var me = this;
+
+            if(me.isTouchDevice()) {
+                event.preventDefault();
+            }
+        },
+
+        /**
+         * Detecting touch device.
+         *
+         * @returns {boolean}
          */
-        onTouchStart: function () {
-            this._touchStartTimestamp = Date.now();
+        isTouchDevice: function() {
+            return true == ("ontouchstart" in window || window.DocumentTouch && document instanceof DocumentTouch);
         },
 
         /**
@@ -191,7 +187,7 @@
 
             event.stopPropagation();
 
-            if (!(event.originalEvent instanceof MouseEvent) && (Date.now() - me._touchStartTimestamp > opts.touchTimeTolerance)) {
+            if (!(event.originalEvent instanceof MouseEvent)) {
                 event.preventDefault();
                 return;
             }