Bug 103305

Summary:	context menu in filebrowser shows only default actions
Product:	[Applications] amarok	Reporter:	Otto Allmendinger <otto.allmendinger>
Component:	general	Assignee:	Amarok Developers <amarok-bugs-dist>
Status:	RESOLVED FIXED
Severity:	normal	CC:	pierre.delagrave
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Gentoo Packages
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Otto Allmendinger 2005-04-05 19:04:30 UTC

Version:           all (using KDE KDE 3.4.0)
Installed from:    Gentoo Packages
Compiler:          gcc (GCC) 3.3.4 20040623 (Gentoo Linux 3.3.4-r1, ssp-3.3.2-2, pie-8.7.6)
 
OS:                Linux

after updating to KDE 3.4, the context menu on the right mouse button is only showing the default actions (parent directory etc) and not "Make Playlist" and friends. Tested with QT 3.3.3 and QT 3.3.4.

Comment 1 Alexandre Oliveira 2005-04-13 00:43:21 UTC

Which Context Menu are you talking about?

Comment 2 Otto Allmendinger 2005-05-03 20:52:33 UTC

look at lines 145-165 in filebrowser.cpp

Comment 3 Alexandre Oliveira 2005-05-16 22:31:42 UTC

Yes, I can confirm this.

Comment 4 Alexandre Oliveira 2005-07-12 02:15:50 UTC

*** Bug 108955 has been marked as a duplicate of this bug. ***

Comment 5 Alexandre Oliveira 2005-08-28 07:54:20 UTC

SVN commit 454132 by aoliveira:

With KDE 3.4, the proper context menu wouldn't be shown for File Browser. 
Patch by Christian Baumgart <christianbaumgart@web.de>
BUG: 103305


 M  +3 -0      ChangeLog  
 M  +6 -0      src/filebrowser.cpp  


--- trunk/extragear/multimedia/amarok/ChangeLog #454131:454132
@@ -55,6 +55,9 @@
       updated and improved.
 
   BUGFIXES:
+    * With KDE 3.4, the proper context menu wouldn't be shown for File
+      Browser. Patch by Christian Baumgart <christianbaumgart@web.de>.
+      (BR 103305)
     * Playcouter and Access Date wouldn't be updated properly for PostgreSQL.
       Patch by Tonton <tonton-lists@team1664.org>. (BR 111519)
     * Clicking twice on the uninstall button for the same script, would make
--- trunk/extragear/multimedia/amarok/src/filebrowser.cpp #454131:454132
@@ -74,6 +74,12 @@
     MyDirOperator( const KURL &url, QWidget *parent ) : KDirOperator( url, parent ) {
         setDirLister( new MyDirLister( true ) );
     }
+public slots:
+    //reimplemented due to a bug in KDirOperator::activatedMenu ( KDE 3.4.2 ) - See Bug #103305
+    virtual void activatedMenu (const KFileItem *item, const QPoint &pos) {
+        updateSelectionDependentActions();
+        ((KActionMenu*)actionCollection()->action("popupMenu"))->popupMenu()->popup( pos );
+    }
 };