From dd89f93181f136c485b42de240feccd5b24e9579 Mon Sep 17 00:00:00 2001 From: Shaun Reich Date: Sun, 1 Mar 2009 02:27:35 +0000 Subject: [PATCH] Fixed a bug in the "Recently Closed Tabs" menu that would cause it to show more recent tabs at the bottom. The most recent tab will now always be shown right below the separator, which is right below the "Clear Recently Closed Tabs" action. svn path=/trunk/KDE/kdebase/apps/; revision=933487 --- src/dolphinmainwindow.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/dolphinmainwindow.cpp b/src/dolphinmainwindow.cpp index 78e489915..fc68191b1 100644 --- a/src/dolphinmainwindow.cpp +++ b/src/dolphinmainwindow.cpp @@ -1355,11 +1355,12 @@ void DolphinMainWindow::rememberClosedTab(int index) action->setData(QVariant::fromValue(closedTab)); action->setIcon(KIcon(iconName)); - //Add our action at the first element, but only do that if it isn't empty, else just append - if (tabsMenu->actions().isEmpty()) { + //Add our action after the separator and the clear list actions + if (tabsMenu->actions().size() == 2) { tabsMenu->addAction(action); } else { - tabsMenu->insertAction(tabsMenu->actions().first() + 2, action); + + tabsMenu->insertAction(tabsMenu->actions().at(2), action); } actionCollection()->action("closed_tabs")->setEnabled(true); } -- 2.47.3