Make QQuick contextmenu actions behave like widget counterparts

Updates a two of the context menu actions so they behave like the updated
actions in qtwebenginewidgets. Also improves the default label of
OpenInThisWindow, since by default there are no other windows.

Change-Id: Iac0dc23e8f31598296cba87f93503cefd926983f
Reviewed-by: Andras Becsi <andras.becsi@theqtcompany.com>

1 files changed, 6 insertions, 3 deletions

diff --git a/src/webengine/api/qquickwebengineview.cpp b/src/webengine/api/qquickwebengineview.cpp
index d9fa735d2..11e4d018c 100644
--- a/src/webengine/api/qquickwebengineview.cpp
+++ b/src/webengine/api/qquickwebengineview.cpp
@@ -179,6 +179,11 @@ bool QQuickWebEngineViewPrivate::contextMenuRequested(const WebEngineContextMenu
     // Populate our menu
     MenuItemHandler *item = 0;
 
+    if (!data.linkText.isEmpty() && data.linkUrl.isValid()) {
+        item = new NavigateMenuItem(menu, adapter, data.linkUrl);
+        ui()->addMenuItem(item, QObject::tr("Follow link"));
+    }
+
     if (data.selectedText.isEmpty()) {
         item = new MenuItemHandler(menu);
         QObject::connect(item, &MenuItemHandler::triggered, q, &QQuickWebEngineView::goBack);
@@ -197,9 +202,7 @@ bool QQuickWebEngineViewPrivate::contextMenuRequested(const WebEngineContextMenu
     }
 
     if (!data.linkText.isEmpty() && data.linkUrl.isValid()) {
-        item = new NavigateMenuItem(menu, adapter, data.linkUrl);
-        ui()->addMenuItem(item, QObject::tr("Navigate to..."));
-        item = new CopyMenuItem(menu, data.linkUrl.toString());
+        item = new CopyLinkMenuItem(menu, data.linkUrl, data.linkText);
         ui()->addMenuItem(item, QObject::tr("Copy link address"));
     }