Bug 130248

Summary:	indicate next repeat option in tooltip
Product:	[Applications] amarok	Reporter:	Stanislav Karchebny <stanislav.karchebny>
Component:	general	Assignee:	Amarok Bugs <amarok-bugs-null>
Status:	RESOLVED FIXED
Severity:	wishlist
Priority:	NOR
Version First Reported In:	1.4.0
Target Milestone:	---
Platform:	Unlisted Binaries
OS:	Linux
Latest Commit:		Version Fixed/Implemented 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;