fix(browser): use button release for forward/back navigation

Also check if mouse pointer is within the webview

Fixes #1564.

Author: trollixx

src/libs/browser/webview.cpp

@@ -206,16 +206,23 @@ void WebView::contextMenuEvent(QContextMenuEvent *event)
     m_contextMenu->popup(event->globalPos());
 }
 
-bool WebView::handleMousePressEvent(QMouseEvent *event)
+bool WebView::handleMouseReleaseEvent(QMouseEvent *event)
 {
     switch (event->button()) {
     case Qt::BackButton:
-        back();
+        // Check if cursor is still inside webview.
+        if (rect().contains(event->pos())) {
+            back();
+        }
+
         event->accept();
         return true;
 
     case Qt::ForwardButton:
-        forward();
+        if (rect().contains(event->pos())) {
+            forward();
+        }
+
         event->accept();
         return true;
 
@@ -251,8 +258,8 @@ bool WebView::eventFilter(QObject *watched, QEvent *event)
 {
     if (watched->parent() == this) {
         switch (event->type()) {
-        case QEvent::MouseButtonPress:
-            if (handleMousePressEvent(static_cast<QMouseEvent *>(event))) {
+        case QEvent::MouseButtonRelease:
+            if (handleMouseReleaseEvent(static_cast<QMouseEvent *>(event))) {
                 return true;
             }
 

src/libs/browser/webview.h

@@ -57,7 +57,7 @@ public slots:
     void contextMenuEvent(QContextMenuEvent *event) override;
 
 private:
-    bool handleMousePressEvent(QMouseEvent *event);
+    bool handleMouseReleaseEvent(QMouseEvent *event);
     bool handleWheelEvent(QWheelEvent *event);
 
     QMenu *m_contextMenu = nullptr;