Color roles, coming this Monday to kdelibs?

Matthew Woehlke mw_triad at users.sourceforge.net
Mon Jun 18 15:57:38 BST 2007


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/





More information about the kde-core-devel mailing list