[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: Color roles, coming this Monday to kdelibs?
From:       Matthew Woehlke <mw_triad () users ! sourceforge ! net>
Date:       2007-06-18 14:57:38
Message-ID: f566h3$9mv$1 () sea ! gmane ! org
[Download RAW message or body]

Olaf Schmidt wrote:
> [ Matthew Woehlke, Mo., 18. Jun. 2007 ]
>> I've no code handy and therefore can't offer an actual patch, but the
>> quickest change is simply to replace: 
>>
>> KGlobalSettings::toolBarHighlightColor()
>> - with -
>>
> KColorScheme(KColorScheme::Button).decoration(KColorScheme::HoverColor).color()
> 
> The colour is here used for the text, so it should instead be 
> KColorScheme(KColorScheme::Button).forground(KColorScheme::ActiveText)

D'oh, that's what I get for replying w/o looking at the source code :-). 
Thanks for keeping me honest, Olaf.

...in which case I *really* don't feel bad nuking
KGS::toolBarHighlightColor(), since the one-and-only user is using it 
incorrectly anyway.

-- 
Matthew
"Still the prettiest." -- Legolas
(as quoted in The Very Secret Diaries by Cassandra Claire)
http://www.ealasaid.com/misc/vsd/

[prev in list] [next in list] [prev in thread] [next in thread]