Version: 2.4.3 (using KDE 4.6.5) OS: Linux Since Amarok 2.4.3 the text in the toolbar is just not readable when you use a light color theme for KDE. In every color theme I have tried the text in the toolbar below the playlist ist white. That's a very big problem because it is just not readable with a lot of color themes. All other text is black - and this one should adapt to the used color theme too. If the background color of that toolbar (I don't know where it takes its color from, it doesn't seem to appear in my color theme) is light - the text of it must be dark. Astonishingly the background color used in that toolbar is also used in the Wikipedia module - and there the text on it is black, how it should be. This should also apply to the toolbar text, for the sake of usability and consistency. Reproducible: Always Steps to Reproduce: Use a light color theme for KDE. Expected Results: Either the text color and the background of the toolbar should respect the KDE colors or at least one of them should be configurable separately in amarok. The text must be readable with any color theme. OS: Linux (i686) release 3.0-1.slh.2-aptosid-686 Compiler: gcc
Created attachment 62618 [details] screenshot: text in toolbar not readable
Created attachment 62619 [details] same background color in wikipedia module, text color black as it should be
Looks like someone didn't understand KColorScheme ;)
That is not the toolbar, but the replacement for the status bar. Bart, could you please look into this?
This "replacement for the status bar" seems to use the foreground color for the selected text but at the same time does *not* use the background color for the selected text. When the color theme is light plus the background color for the selected text is dark and the foreground color for the selected text therefore is light in the theme - then amarok has a problem (i.e. its user). The background color for the status bar replacement bar is some mixture of some other color settings?
Re-add CC members that got lost with last change.
(In reply to comment #6) > Re-add CC members that got lost with last change. Why? I removed them as they are not maintaining this at all, so there is no point to subscribe them, instead subscribing the developer actually handling the code makes much more sense.
> Why? Sorry, I forgot to read the activity backlog, just assumed they had added themselves to this bug.
Still valid with current git, very annoying, there should be more contrast here.
From the usability perspective, the situation is clear: If text in an application is not readable in certain color schemes (unless the scheme itself is broken), we have a bug that should be fixed by using the color scheme correctly.
I tested it with a lot of different color themes and it seems to be ok for most themes now. There are very few for which the contrast between background and text color isn't optimal because the color for the selected background seems a little lightened in that bar. Might be considered as fixed?
Git commit 8d3faf37cc2283e5d4038e864adb2343961f1470 by Mark Kretschmann. Committed on 01/07/2013 at 05:46. Pushed by markey into branch 'master'. Text color in toolbar below playlist is no longer always white. Patch by Ricardo Varas <ricardoivaras@gmail.com>. FIXED-IN: 2.8 REVIEW: 111334 Thanks for the patch, Ricardo :) Now, if you could also figure out how to make the text stick less close to the right border, that'd be gold. M +1 -1 src/playlist/PlaylistDock.cpp http://commits.kde.org/amarok/8d3faf37cc2283e5d4038e864adb2343961f1470