Merge pull request #1124 from nvask/fix-mouse-back-forward-double-action

Fix mouse back/forward buttons running action twice (#1115)

Author: andrewrabert

src/ui/EventFilter.cpp

@@ -101,18 +101,23 @@ bool EventFilter::eventFilter(QObject* watched, QEvent* event)
       }
     }
 
-    if (event->type() == QEvent::MouseButtonPress)
+    if (event->type() == QEvent::MouseButtonPress || event->type() == QEvent::MouseButtonRelease)
     {
       QMouseEvent* mouseEvent = dynamic_cast<QMouseEvent*>(event);
 
-      if (mouseEvent) {
-        QQuickItem* webView = window->findChild<QQuickItem*>("web");
-
-        if (mouseEvent->button() == Qt::BackButton)
-          QMetaObject::invokeMethod(webView, "goBack");
+      if (mouseEvent && (mouseEvent->button() == Qt::BackButton || mouseEvent->button() == Qt::ForwardButton)) {
+        // Only trigger navigation on release, but block both press and release
+        // to prevent WebEngine from interpreting press-without-release as a long-press
+        if (event->type() == QEvent::MouseButtonRelease)
+        {
+          QQuickItem* webView = window->findChild<QQuickItem*>("web");
 
-        if (mouseEvent->button() == Qt::ForwardButton)
-          QMetaObject::invokeMethod(webView, "goForward");
+          if (mouseEvent->button() == Qt::BackButton)
+            QMetaObject::invokeMethod(webView, "goBack");
+          else
+            QMetaObject::invokeMethod(webView, "goForward");
+        }
+        return true;
       }
     }