Bug 304043

Summary: Please make the dynamic playlist notification bar more readable
Product: [Applications] amarok Reporter: Gandalf Lechner <gandalflechner>
Component: PlaylistAssignee: Amarok Developers <amarok-bugs-dist>
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:
Sentry Crash Report:
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]
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().

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


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