[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:       Olaf Schmidt <ojschmidt () kde ! org>
Date:       2007-06-18 9:38:14
Message-ID: 200706181138.16218.ojschmidt () kde ! org
[Download RAW message or body]

[ 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)

The change would be in line 811 of
http://lxr.kde.org/source/koffice/libs/guiutils/KoTooluButton.cpp#811 :

      // Draw the text at the position given by textRect, and using textFlags
      if (!textLabel().isNull() && !textRect.isNull()) {
          _painter->setFont( KGlobalSettings::toolBarFont() );
          if ( !isEnabled() )
              _painter->setPen( palette().disabled().dark() );
          else if( KToolBarButton::isRaised() )
-             _painter->setPen( KGlobalSettings::toolBarHighlightColor() );
+             _painter->setBrush( 
KColorScheme(KColorScheme::Button).forground(KColorScheme::ActiveText) );
          else
              _painter->setPen( colorGroup().buttonText() );
          _painter->drawText( textRect, textFlags, textLabel() );


(No real diff, sorry.)

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