To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=48609





------- Additional comments from [EMAIL PROTECTED] Tue May  3 05:47:05 -0700 
2005 -------
So, the theme colors are hardcoded in the themes :-( 
 
OK to commit the following? 
 
--- unx/kde/salnativewidgets-kde.cxx    2 May 2005 13:46:21 -0000       1.6.4.6 
+++ unx/kde/salnativewidgets-kde.cxx    3 May 2005 12:43:51 -0000 
@@ -1956,10 +1956,24 @@ void KDESalFrame::UpdateSettings( AllSet 
         aStyleSettings.SetMenuColor( toColor( qMenuCG.button() ) ); 
         aStyleSettings.SetMenuBarColor( toColor( qMenuCG.button() ) ); 
         aStyleSettings.SetMenuHighlightColor( toColor 
( qMenuCG.highlight() ) ); 
-        aStyleSettings.SetMenuHighlightTextColor( toColor 
( qMenuCG.highlightedText() ) ); 
+ 
+        // Menu items higlight text color, theme specific 
+        if ( kapp->style().inherits( "HighContrastStyle" ) || 
+                kapp->style().inherits( "KeramikStyle" ) || 
+                kapp->style().inherits( "QWindowsStyle" ) || 
+                kapp->style().inherits( "ThinKeramikStyle" ) || 
+                kapp->style().inherits( "PlastikStyle" ) ) 
+        { 
+            aStyleSettings.SetMenuHighlightTextColor( toColor 
( qMenuCG.highlightedText() ) ); 
+        } 
+        else 
+            aStyleSettings.SetMenuHighlightTextColor( toColor 
( qMenuCG.buttonText() ) ); 
 
         // set special menubar higlight text color 
-        ImplGetSVData()->maNWFData.maMenuBarHighlightTextColor = 
toColor( qMenuCG.buttonText() ); 
+        if ( kapp->style().inherits( "HighContrastStyle" ) ) 
+            ImplGetSVData()->maNWFData.maMenuBarHighlightTextColor = 
toColor( qMenuCG.highlightedText() ); 
+        else 
+            ImplGetSVData()->maNWFData.maMenuBarHighlightTextColor = 
toColor( qMenuCG.buttonText() ); 
 
         // Font 
         Font aFont = toFont( pMenuBar->font(), rSettings.GetUILocale() ); 
 

---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to