From 31fc08fe7ff06d8660624a360a52c7f9af2854a4 Mon Sep 17 00:00:00 2001 From: Loren Burkholder Date: Fri, 7 Apr 2023 10:59:05 -0400 Subject: [PATCH 1/1] Move popout action into split action dropdown --- src/dolphinmainwindow.cpp | 43 +++++++++++++++++++++------------------ src/dolphinmainwindow.h | 2 ++ src/dolphinui.rc | 1 - 3 files changed, 25 insertions(+), 21 deletions(-) diff --git a/src/dolphinmainwindow.cpp b/src/dolphinmainwindow.cpp index 4a327182e..af217eab4 100644 --- a/src/dolphinmainwindow.cpp +++ b/src/dolphinmainwindow.cpp @@ -118,6 +118,7 @@ DolphinMainWindow::DolphinMainWindow() , m_sessionSaveTimer(nullptr) , m_sessionSaveWatcher(nullptr) , m_sessionSaveScheduled(false) + , m_splitViewAction(nullptr) { Q_INIT_RESOURCE(dolphin); @@ -1830,14 +1831,15 @@ void DolphinMainWindow::setupActions() // setup 'View' menu // (note that most of it is set up in DolphinViewActionHandler) - QAction *split = actionCollection()->addAction(QStringLiteral("split_view")); - split->setWhatsThis(xi18nc("@info:whatsthis find", - "This splits " - "the folder view below into two autonomous views.This " - "way you can see two locations at once and move items between them " - "quickly.Click this again afterwards to recombine the views.")); - actionCollection()->setDefaultShortcut(split, Qt::Key_F3); - connect(split, &QAction::triggered, this, &DolphinMainWindow::toggleSplitView); + m_splitViewAction = actionCollection()->add(QStringLiteral("split_view")); + m_splitViewAction->setWhatsThis(xi18nc("@info:whatsthis find", + "This splits " + "the folder view below into two autonomous views.This " + "way you can see two locations at once and move items between them " + "quickly.Click this again afterwards to recombine the views.")); + m_splitViewAction->setPopupMode(QToolButton::MenuButtonPopup); + actionCollection()->setDefaultShortcut(m_splitViewAction, Qt::Key_F3); + connect(m_splitViewAction, &QAction::triggered, this, &DolphinMainWindow::toggleSplitView); QAction *popoutSplit = actionCollection()->addAction(QStringLiteral("popout_split_view")); popoutSplit->setWhatsThis(xi18nc("@info:whatsthis", @@ -1845,6 +1847,7 @@ void DolphinMainWindow::setupActions() "view out into a new window.")); popoutSplit->setIcon(QIcon::fromTheme(QStringLiteral("window-new"))); actionCollection()->setDefaultShortcut(popoutSplit, Qt::SHIFT | Qt::Key_F3); + m_splitViewAction->addAction(popoutSplit); connect(popoutSplit, &QAction::triggered, this, &DolphinMainWindow::popoutSplitView); QAction *stashSplit = actionCollection()->addAction(QStringLiteral("split_stash")); @@ -2525,29 +2528,29 @@ void DolphinMainWindow::connectViewSignals(DolphinViewContainer *container) void DolphinMainWindow::updateSplitActions() { - QAction *splitAction = actionCollection()->action(QStringLiteral("split_view")); QAction *popoutSplitAction = actionCollection()->action(QStringLiteral("popout_split_view")); const DolphinTabPage *tabPage = m_tabWidget->currentTabPage(); if (tabPage->splitViewEnabled()) { if (GeneralSettings::closeActiveSplitView() ? tabPage->primaryViewActive() : !tabPage->primaryViewActive()) { - splitAction->setText(i18nc("@action:intoolbar Close left view", "Close")); - splitAction->setToolTip(i18nc("@info", "Close left view")); - splitAction->setIcon(QIcon::fromTheme(QStringLiteral("view-left-close"))); + m_splitViewAction->setText(i18nc("@action:intoolbar Close left view", "Close")); + m_splitViewAction->setToolTip(i18nc("@info", "Close left view")); + m_splitViewAction->setIcon(QIcon::fromTheme(QStringLiteral("view-left-close"))); popoutSplitAction->setText(i18nc("@action:intoolbar Move left split view to a new window", "Pop out")); popoutSplitAction->setToolTip(i18nc("@info", "Move left split view to a new window")); } else { - splitAction->setText(i18nc("@action:intoolbar Close right view", "Close")); - splitAction->setToolTip(i18nc("@info", "Close right view")); - splitAction->setIcon(QIcon::fromTheme(QStringLiteral("view-right-close"))); + m_splitViewAction->setText(i18nc("@action:intoolbar Close right view", "Close")); + m_splitViewAction->setToolTip(i18nc("@info", "Close right view")); + m_splitViewAction->setIcon(QIcon::fromTheme(QStringLiteral("view-right-close"))); popoutSplitAction->setText(i18nc("@action:intoolbar Move right split view to a new window", "Pop out")); popoutSplitAction->setToolTip(i18nc("@info", "Move right split view to a new window")); } - popoutSplitAction->setVisible(true); + popoutSplitAction->setEnabled(true); } else { - splitAction->setText(i18nc("@action:intoolbar Split view", "Split")); - splitAction->setToolTip(i18nc("@info", "Split view")); - splitAction->setIcon(QIcon::fromTheme(QStringLiteral("view-right-new"))); - popoutSplitAction->setVisible(false); + m_splitViewAction->setText(i18nc("@action:intoolbar Split view", "Split")); + m_splitViewAction->setToolTip(i18nc("@info", "Split view")); + m_splitViewAction->setIcon(QIcon::fromTheme(QStringLiteral("view-right-new"))); + popoutSplitAction->setText(i18nc("@action:intoolbar Move active split view to a new window", "Pop out")); + popoutSplitAction->setEnabled(false); } } diff --git a/src/dolphinmainwindow.h b/src/dolphinmainwindow.h index 844360aaa..7360fffbf 100644 --- a/src/dolphinmainwindow.h +++ b/src/dolphinmainwindow.h @@ -12,6 +12,7 @@ #include "config-dolphin.h" #include "dolphintabwidget.h" #include "selectionmode/bottombar.h" +#include #include #include #include @@ -735,6 +736,7 @@ private: KToolBarPopupAction *m_backAction; KToolBarPopupAction *m_forwardAction; + KActionMenu *m_splitViewAction; QMenu m_searchTools; KFileItemActions m_fileItemActions; diff --git a/src/dolphinui.rc b/src/dolphinui.rc index 7d35ed303..05eb33469 100644 --- a/src/dolphinui.rc +++ b/src/dolphinui.rc @@ -112,7 +112,6 @@ - -- 2.47.3