Bug 304043

Summary:	Please make the dynamic playlist notification bar more readable
Product:	[Applications] amarok	Reporter:	Gandalf Lechner <gandalflechner>
Component:	Playlist	Assignee:	Amarok Developers <amarok-bugs-dist>
Status:	RESOLVED DUPLICATE
Severity:	normal	CC:	teo
Priority:	NOR
Version:	2.5.90 (2.6 beta)
Target Milestone:	2.6
Platform:	Ubuntu
OS:	Linux
Latest Commit:		Version Fixed In:
Attachments:	screeenshot

Description Gandalf Lechner 2012-07-25 08:17:49 UTC

In some recent version, a slim bar/widget at the top of the playlist was introduced to control the dynamic playlists feature, with buttons to switch it off and to repopulate. That is really cool and actually something I wanted for a long time =) . 

However, currently it is designed in medium blue on dark blue and can therefore basically only be operated either by amarok instinct or eagle eyes.. see screenshot. Any chance of changing the colors to something more readable?

Reproducible: Always

Comment 1 Gandalf Lechner 2012-07-25 08:18:55 UTC

Created attachment 72749 [details]
screeenshot

Comment 2 Myriam Schweingruber 2012-07-25 08:29:29 UTC

This is already fixed in the upcoming Amarok 2.6 by this commit:

Git commit e17bd07d9e209e0511b136be1f5c31516a558764 by Matěj Laitl.
Committed on 09/06/2012 at 15:17.
Pushed by laitl into branch 'master'.

Use proper color in "Dynamic Playlist" notification strip

...it was not using style's link colour, from QPalette documentation:
> Note that we do not use the Link and LinkVisited roles when rendering
> rich text in Qt, and that we recommend that you use CSS

I didn't know this.

This should fix low-contrast problem with Obsidian Coast color scheme
and the like. It is still not optimal, but that would require
rethinking PaletteHandler::highlightColor().
FIXED-IN: 2.6

M  +1    -0    ChangeLog
M  +5    -3    src/playlist/PlaylistDock.cpp

http://commits.kde.org/amarok/e17bd07d9e209e0511b136be1f5c31516a558764

*** This bug has been marked as a duplicate of bug 301303 ***