Bug 263330 - trayicon menu: quit not the last option if you play a song
Summary: trayicon menu: quit not the last option if you play a song
Status: RESOLVED FIXED
Alias: None
Product: amarok
Classification: Applications
Component: general (show other bugs)
Version: 2.4.0
Platform: openSUSE Linux
: NOR normal
Target Milestone: 2.4.1
Assignee: Amarok Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-16 17:09 UTC by Javier Llorente
Modified: 2011-01-16 20:50 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 2.4.1
Sentry Crash Report:


Attachments
tray icon menu after playing a song (16.84 KB, image/png)
2011-01-16 17:09 UTC, Javier Llorente
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Javier Llorente 2011-01-16 17:09:58 UTC
Created attachment 56096 [details]
tray icon menu after playing a song

Version:           2.4.0 (using KDE 4.5.5) 
OS:                Linux



Reproducible: Always

Steps to Reproduce:
1. Run Amarok
2. Right click on Amarok's trayicon. Everything looks OK.
3. Play a song
4. Right click again on the trayicon

Actual Results:  
"Configure Amarok" is the last option

Expected Results:  
"Quit" should be the last one
Comment 1 Javier Llorente 2011-01-16 17:13:12 UTC
Please see attached screenshot ;-)
This happens on openSUSE 11.3 (64 bit), with KDE 4.5.5 and Qt 4.6.3
Comment 2 Javier Llorente 2011-01-16 18:29:22 UTC
It also affects Amarok 2.4.0 on KDE 4.5.95/Qt 4.7.1
Comment 3 Sergey Ivanov 2011-01-16 19:22:06 UTC
commit bf2d34aff5b343a331eca52b60fd9edb74d8ebfe
branch master
Author: Sergey Ivanov <123kash@gmail.com>
Date:   Sun Jan 16 21:21:27 2011 +0300

    Fixed wrong "Configure amarok" action position in tray icon popup menu.
    BUG: 263330

diff --git a/ChangeLog b/ChangeLog
index 9147c4e..7736b92 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -13,6 +13,7 @@ VERSION 2.4.1
       <bhargav.1191@gmail.com>. (BR 248690)
 
   BUGFIXES:
+    * Fixed wrong "Configure amarok" action position in tray icon popup menu. (BR 263330)
     * Fixed minor inconsistency in the Collection view. Patch by 
       Bhargav Mangipudi <bhargav.1191@gmail.com>. (BR 260973)
     * Fixed potential crash with the Similar Artists applet. Thanks to 
diff --git a/src/TrayIcon.cpp b/src/TrayIcon.cpp
index f9cb8d7..768424e 100644
--- a/src/TrayIcon.cpp
+++ b/src/TrayIcon.cpp
@@ -296,9 +296,9 @@ Amarok::TrayIcon::updateMenu()
 
         m_separator = contextMenu()->addSeparator();
         // readd
+        contextMenu()->addAction( preferenceAction );
         contextMenu()->addAction( actionCollection()->action( "minimizeRestore" ) );
         contextMenu()->addAction( actionCollection()->action( "file_quit" ) );
-        contextMenu()->addAction( preferenceAction );
     }
 }