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

List:       kde-cygwin
Subject:    Re: [PATCH] for Borland & widget-colors
From:       Holger Schroeder <holger-kde () holgis ! net>
Date:       2004-09-30 21:05:58
Message-ID: 200409302305.58765.holger-kde () holgis ! net
[Download RAW message or body]

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

_______________________________________________
kde-cygwin mailing list
kde-cygwin@kde.org
https://mail.kde.org/mailman/listinfo/kde-cygwin
[prev in list] [next in list] [prev in thread] [next in thread]