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

List:       kde-commits
Subject:    KDE/kdebase/workspace/systemsettings
From:       Christoph Feck <christoph () maxiom ! de>
Date:       2009-04-14 22:47:01
Message-ID: 1239749221.469255.13845.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 954033 by cfeck:

Providing a good default size is hard, forcing a minimum size is evil


 M  +2 -4      mainwindow.cpp  
 M  +0 -1      systemsettingsui.rc  


--- trunk/KDE/kdebase/workspace/systemsettings/mainwindow.cpp #954032:954033
@@ -69,13 +69,10 @@
     moduleTabs = new KTabWidget(this);
     moduleTabs->setDocumentMode(true);
 
-    // Set a minimum size to avoid double scrollbars ... at least try.
-    setMinimumSize(800,480);
-
     buildActions();
     buildMainWidget();
     // We hide the menubar. So ensure the toolbar is always visible because you cannot get it back
-    setupGUI(Save|Create,QString());
+    setupGUI(QSize(800, 480), Save | Create, QString());
     menuBar()->hide();
 
     search->setFocus();
@@ -242,6 +239,7 @@
     searchLabel->setFont(KGlobalSettings::toolBarFont());
     QHBoxLayout * hlay = new QHBoxLayout( searchWid );
     hlay->setMargin(0);
+    hlay->addSpacing( KDialog::spacingHint() );
     hlay->addWidget( searchIcon );
     hlay->addWidget( searchLabel );
     hlay->addSpacing( KDialog::spacingHint() );
--- trunk/KDE/kdebase/workspace/systemsettings/systemsettingsui.rc #954032:954033
@@ -4,7 +4,6 @@
 <ToolBar name="mainToolBar" iconText="icontextright">
 	<Action name="go_back"/>
 	<Separator/>
-	<Action name="spacer"/>
 	<Action name="searchText"/>
 	<Action name="search"/>
 </ToolBar>
[prev in list] [next in list] [prev in thread] [next in thread]