Bug 130248

Summary: indicate next repeat option in tooltip
Product: [Applications] amarok Reporter: Stanislav Karchebny <stanislav.karchebny>
Component: generalAssignee: Amarok Developers <amarok-bugs-dist>
Status: RESOLVED FIXED    
Severity: wishlist    
Priority: NOR    
Version: 1.4.0   
Target Milestone: ---   
Platform: Unlisted Binaries   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Stanislav Karchebny 2006-07-04 15:24:36 UTC 
Version:           1.4.0 (using KDE KDE 3.5.3)
Installed from:    Unspecified Linux
Compiler:          gcc 4.1 
OS:                Linux

Right now tooltip for Repeat option in statusbar only shows current state, it would be nice usability-wise to show what state it will become after clicking:

Now:
+----------------+
| Repeat: off    |
+----------------+

Then:
+--------------------+
| Repeat: off        |
|                    |
| Next: Repeat track |
|                    |
| Click to change.   |
+--------------------+
Comment 1 Alexandre Oliveira 2006-12-27 20:50:27 UTC 
Sorry, showing the next too much, but the "click to change" is a good idea.
Comment 2 Alexandre Oliveira 2006-12-27 20:55:22 UTC 
SVN commit 617030 by aoliveira:

Add a "Click to change" to the KSelectAction's tooltips of playlist window
BUG: 130248


 M  +3 -0      actionclasses.cpp  
 M  +2 -0      actionclasses.h  


--- trunk/extragear/multimedia/amarok/src/actionclasses.cpp #617029:617030
@@ -411,6 +411,9 @@
     return QString::null;
 }
 
+QString SelectAction::currentText() const {
+    return KSelectAction::currentText() + "<br /><br />" + i18n("Click to change");
+}
 
 //////////////////////////////////////////////////////////////////////////////////////////
 // VolumeAction
--- trunk/extragear/multimedia/amarok/src/actionclasses.h #617029:617030
@@ -120,6 +120,8 @@
 
             virtual void setIcons( QStringList icons );
 
+            virtual QString currentText() const;
+
             QStringList icons() const;
 
             QString currentIcon() const;