Bug 304043 - Please make the dynamic playlist notification bar more readable
Summary: Please make the dynamic playlist notification bar more readable
Status: RESOLVED DUPLICATE of bug 301303
Alias: None
Product: amarok
Classification: Applications
Component: Playlist (show other bugs)
Version: 2.5.90 (2.6 beta)
Platform: Ubuntu Linux
: NOR normal
Target Milestone: 2.6
Assignee: Amarok Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-25 08:17 UTC by Gandalf Lechner
Modified: 2012-07-25 08:29 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
screeenshot (18.27 KB, image/png)
2012-07-25 08:18 UTC, Gandalf Lechner
Details

Note You need to log in before you can comment on or make changes to this bug.
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 ***