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/webview.cpp') 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/webview.cpp') 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