[PATCH] making the kicker quickbrowser even quicker (usability wise) ;-)

Alexander Neundorf alexander.neundorf at gmx.net
Wed Jan 29 21:03:30 GMT 2003


Hi,

the following patch places the "Open in filemanager" and "Open in terminal" as 
the first two items in the quickbrowser menu instead of an own popup menu -> 
saves a lot of clicking and enlarges the menu only by one item :-)

Any objections ?

Bye
Alex

RCS file: /home/kde/kdebase/kicker/ui/browser_mnu.cpp,v
retrieving revision 1.45
diff -b -u -p -r1.45 browser_mnu.cpp
--- browser_mnu.cpp     15 Jan 2003 05:02:03 -0000      1.45
+++ browser_mnu.cpp     29 Jan 2003 21:00:10 -0000
@@ -141,6 +141,15 @@ void PanelBrowserMenu::initialize()
        return;
     }

+    // insert file manager and terminal entries
+    // only the first part menu got them
+    if(_startid == 0) {
+       insertItem(CICON("kfm"), i18n("Open in File Manager"), this, 
SLOT(slotOpenFileManager()));
+        if (kapp->authorize("shell_access"))
+            insertItem(CICON("terminal"), i18n("Open in Terminal"), this, 
SLOT(slotOpenTerminal()));
+    }
+
+
     bool first_entry = true;
     bool dirfile_separator = false;
     int item_count = 0;
@@ -307,19 +316,6 @@ void PanelBrowserMenu::initialize()
         dirname.prepend("...");
     }
     setCaption(dirname);
-
-    // insert file manager and terminal entries
-    // only the first part menu got them
-    if(_startid == 0) {
-        QPopupMenu *opmenu = new QPopupMenu(this);
-
-        dirname.replace(QRegExp("&"), "&&");
-        insertItem(CICON("kdisknav"), dirname, opmenu, -1, 0);
-
-        opmenu->insertItem(CICON("kfm"), i18n("Open in File Manager"), this, 
SLOT(slotOpenFileManager()));
-        if (kapp->authorize("shell_access"))
-            opmenu->insertItem(CICON("terminal"), i18n("Open in Terminal"), 
this, SLOT(slotOpenTerminal()));
-    }

     // setup and start delayed mimetype check timer
     if(_mimemap.count() > 0) {





More information about the kde-core-devel mailing list