UI: wrong values in amarok's theme color scheme

Martin darklight.xdarklight at googlemail.com
Mon Aug 9 21:48:36 CEST 2010


Hi,

as promised: here's the update-email with the patches attached.
I've also pushed the code to my clone at gitorious - see here: [0].

Here's a list of changes:
-small code cleanup for lyrics related code
-the lyrics in the LyricsApplet are now updated if the user changes them in 
TagDialog
-due to change #2: the user now gets a warning when some of his changes he 
made to the lyrics in the LyricsApplet might get lost (see a screenshot of it 
here: [1]) -> this fixes bug 207621 ([3])
-I also ported the other KMessageBox (which showed up when the user clicked 
the "refresh" button) to Plasma::Applet's showMessage()

Any comments regarding my patches are welcome :)

PS: I have commit-access to amarok's git repository.
If you want I will commit the changes myself (and resolve any conflicts if 
necessary).

Regards,
Martin

[0] http://gitorious.org/~xdarklight/amarok/xdarklights-clone/commits/improve-
lyrics-handling
[1] http://www.abload.de/img/amarokscreenprh2.png
[3] https://bugs.kde.org/show_bug.cgi?id=207621
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amarok-improve-lyrics-handling.tar.bz
Type: application/x-bzip-compressed-tar
Size: 7380 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/amarok-devel/attachments/20100809/e64f6294/attachment-0001.tbz2 


More information about the Amarok-devel mailing list