diff options
| author | Lindsay Mathieson <lindsay.mathieson@gmail.com> | 2013-04-25 11:53:27 +1000 |
|---|---|---|
| committer | Lindsay Mathieson <lindsay.mathieson@gmail.com> | 2013-04-25 11:53:27 +1000 |
| commit | d116649be7920bc04d27f0096f00f6247e6b293b (patch) | |
| tree | c193bc69b88f9768b6b5495e5402228277f30e05 | |
| parent | Favorites (fast) management, the "opera" way (diff) | |
| download | rekonq-d116649be7920bc04d27f0096f00f6247e6b293b.tar.xz | |
Changes order of link actions for link context menu
Moved "Open Here" action (if needed) to after "Open in Private Window"
so as to keep the order of link actions consistent. This reduces the
need for users to inspect the context menu when right clicking links.
REVIEW: 110168
| -rw-r--r-- | src/webtab/webview.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/webtab/webview.cpp b/src/webtab/webview.cpp index 44b24cc5..df91451f 100644 --- a/src/webtab/webview.cpp +++ b/src/webtab/webview.cpp @@ -380,14 +380,6 @@ void WebView::contextMenuEvent(QContextMenuEvent *event) sendByMailAction->setText(i18n("Share link")); - if (m_isExternalLinkHovered) - { - a = new KAction(KIcon("view-close"), i18n("Open &Here"), this); - a->setData(m_contextMenuHitResult.linkUrl()); - connect(a, SIGNAL(triggered(bool)), this, SLOT(openLinkHere())); - menu.addAction(a); - } - a = new KAction(KIcon("tab-new"), i18n("Open in New &Tab"), &menu); a->setData(m_contextMenuHitResult.linkUrl()); connect(a, SIGNAL(triggered(bool)), this, SLOT(openLinkInNewTab())); @@ -406,6 +398,14 @@ void WebView::contextMenuEvent(QContextMenuEvent *event) menu.addAction(a); } + if (m_isExternalLinkHovered) + { + a = new KAction(KIcon("view-close"), i18n("Open &Here"), this); + a->setData(m_contextMenuHitResult.linkUrl()); + connect(a, SIGNAL(triggered(bool)), this, SLOT(openLinkHere())); + menu.addAction(a); + } + menu.addSeparator(); // Don't show dots if we are NOT going to ask for download path |