[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdebase/apps/dolphin/src
From:       Shaun Reich <shaun.reich () kdemail ! net>
Date:       2009-03-01 2:27:35
Message-ID: 1235874455.545559.16451.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 933487 by sreich:

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.

 M  +4 -3      dolphinmainwindow.cpp  


--- trunk/KDE/kdebase/apps/dolphin/src/dolphinmainwindow.cpp #933486:933487
@@ -1355,11 +1355,12 @@
     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);
 }


[prev in list] [next in list] [prev in thread] [next in thread]