[PATCH] for Borland & widget-colors

Holger Schroeder holger-kde at holgis.net
Thu Sep 30 23:05:58 CEST 2004


Hi,

was this really intended ?

<snip>

+    if ( !( pal == QApplication::palette() ) )
+        QApplication::setPalette( pal );
+
+    // now setup the palette for special widgets
+    // only set the values which are different from standard values...
+
+    //Popup Menu
+    pal = QApplication::palette();
+    qc = getQColorFromSys( COLOR_MENU );
     pal.setBrush( QPalette::Active, QColorGroup::Background, qc );
     pal.setBrush( QPalette::Inactive, QColorGroup::Background, qc );
-    pal.setBrush( QPalette::Disabled, QColorGroup::Background, qc );
+    pal.setBrush( QPalette::Inactive, QColorGroup::Background, qc );

so you will end up with setting the same thing twice in the two last lines...


Holger

On Thursday 30 September 2004 22:33, Christian Ehrlicher wrote:
> Today I've two patches
>
> 1. standard color definitions: since some of them were not set correctly
> (see "[PATCH] Correct color for disabled Buttons") I tried to set all to
> the correct value. Feel free to correct them if you find an error :)
>
> 2. A fix for the "undefined reference to qMain in qtmain.lib" - problem
> when using Borland. First we only need qMain in a gui-application
> (qwindowdefs_win.h). The change in qtmain.cpp was a try and error
> solution ... but it works :)
>
> Christian



More information about the kde-cygwin mailing list