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

List:       kde-commits
Subject:    KDE/kdelibs/kdeui/kernel
From:       Jaroslaw Staniek <js () iidea ! pl>
Date:       2008-11-13 8:29:15
Message-ID: 1226564955.032816.8265.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 883540 by staniek:

Revert Aaron's r664813 change (for kapplication.cpp only) to enable -style option on \
!X11 targets too. 

Reviewed by Aaron.

Aaron's commit says
"only set the style when we're on x11; let Qt handle the style and palette selection \
on the other systems. if people wish to change the style of Qt apps on those \
platforms, they can use qtconfig"

But from http://doc.trolltech.com/4.4/qtconfig.html:

"qtconfig is available on X11 platforms and should be installed alongside the other \
tools supplied with Qt."


I and possibly others need the quick --style option 
during the development for various style/appearance-related 
tests on the non-X11 targets.




 M  +0 -2      kapplication.cpp  


--- trunk/KDE/kdelibs/kdeui/kernel/kapplication.cpp #883539:883540
@@ -846,7 +846,6 @@
 {
     KCmdLineArgs *args = KCmdLineArgs::parsedArgs("kde");
 
-#ifdef Q_WS_X11
     if (args && args->isSet("style"))
     {
         extern QString kde_overrideStyle; // see KGlobalSettings. Should we have a \
static setter? @@ -856,7 +855,6 @@
         else
             qWarning() << i18n("The style '%1' was not found", reqStyle);
     }
-#endif
 
     if ( q->type() != KApplication::Tty ) {
         if (args && args->isSet("icon"))


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