SVN commit 854035 by menard:

Fix a bug with the color in tab, the Qt::Black is obviously a bad thing, but we 
have a plasma theme that provide us colors, in my case widgets background are 
blacks in all KDE so the text color provided by plasma::theme can't works.
In all cases the theme color choice by plasma can't works. Using plasma theme 
in tabbar is strange because, just inside in kickoff we use the 
plasma::delegate which use the global colorscheme, it means inconsistency 
beetween inactive colors in plasma::theme and global inactive colors for 
example ; it's not problem in other places but in the same "application". Need 
feedbacks.

CCMAIL:plasma-devel@kde.org


 M  +2 -2      tabbar.cpp  


--- trunk/KDE/kdebase/workspace/plasma/applets/kickoff/ui/tabbar.cpp 
#854034:854035
@@ -300,9 +300,9 @@
         // draw tab text
         //TODO: we may want a nice animation also for the text
         if (i != currentTab){
-            painter.setPen(KColorScheme(QPalette::Active, KColorScheme::View, 
Plasma::Theme::defaultTheme()->colorScheme()).foreground().color());
+            
painter.setPen(QPen(KColorScheme(QPalette::Active).foreground(KColorScheme::InactiveText),
 1));
         }else{
-            painter.setPen(Qt::black); //FIXME: we musn't use hardcoded color 
values
+            
painter.setPen(QPen(KColorScheme(QPalette::Active).foreground(KColorScheme::NormalText),
 1));
         }
         QRect textRect = rect;
         textRect.setTop(textRect.bottom() - textHeight);
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to