From d9d892d4220036ac3e3c8ca1f3a0d50074deff60 Mon Sep 17 00:00:00 2001 From: matgic78 Date: Sun, 18 Oct 2009 12:49:05 +0200 Subject: Add an icon to the "search with" contextmenu action --- src/webview.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src') diff --git a/src/webview.cpp b/src/webview.cpp index 9e06c9da..20058ce1 100644 --- a/src/webview.cpp +++ b/src/webview.cpp @@ -168,7 +168,7 @@ void WebView::contextMenuEvent(QContextMenuEvent *event) // is content selected? Add SEARCH actions if(result.isContentSelected()) { - KActionMenu *searchMenu = new KActionMenu(i18n("Search with"), this); + KActionMenu *searchMenu = new KActionMenu(KIcon("edit-find"), i18n("Search with"), this); KConfig config("kuriikwsfilterrc"); //Share with konqueror KConfigGroup cg = config.group("General"); -- cgit v1.2.1 From e216d455ff141949c17977e29ec7e0392f20863f Mon Sep 17 00:00:00 2001 From: matgic78 Date: Sun, 18 Oct 2009 12:52:13 +0200 Subject: add a separator beetween text and link actions in contextmenu --- src/webview.cpp | 2 ++ 1 file changed, 2 insertions(+) (limited to 'src') diff --git a/src/webview.cpp b/src/webview.cpp index 9e06c9da..25b7f8ea 100644 --- a/src/webview.cpp +++ b/src/webview.cpp @@ -135,6 +135,8 @@ void WebView::contextMenuEvent(QContextMenuEvent *event) a = pageAction(QWebPage::CopyLinkToClipboard); a->setIcon(KIcon("edit-copy")); menu.addAction(a); + + menu.addSeparator(); } // is content editable && selected? Add CUT -- cgit v1.2.1 From 4f37ee951857a68bb58aff32ed2475002997d10c Mon Sep 17 00:00:00 2001 From: matgic78 Date: Sun, 18 Oct 2009 15:17:18 +0200 Subject: set text for copy action in contexmenu to "Copy Text" to avoid confusion with copy link --- src/webview.cpp | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'src') diff --git a/src/webview.cpp b/src/webview.cpp index 9e06c9da..5e5fdbb1 100644 --- a/src/webview.cpp +++ b/src/webview.cpp @@ -153,6 +153,10 @@ void WebView::contextMenuEvent(QContextMenuEvent *event) a = pageAction(QWebPage::Copy); a->setIcon(KIcon("edit-copy")); a->setShortcut(KStandardShortcut::copy().primary()); + if(!result.isContentEditable()) // "Cut" "Copy Text" "Paste" is ugly. Don't add "text" with cut/paste + a->setText(i18n("Copy Text")); + else + a->setText(i18n("Copy")); menu.addAction(a); } -- cgit v1.2.1