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

List:       kde-commits
Subject:    RE: KDE/kdelibs/kdeui/colors
From:       "Matthew Woehlke" <mwoehlke () tibco ! com>
Date:       2007-12-10 18:17:58
Message-ID: E4B46A50CCA52348A67B3506C79BB390013B7107 () NA-PA-VBE01 ! na ! tibco ! com
[Download RAW message or body]

Is this intended to make the button smaller than the icon (which it
does)? It looks really strange...

-- 
Matthew

-----Original Message-----
From: Urs Wolfer [mailto:uwolfer@kde.org] 
Sent: Sunday, December 09, 2007 15:18
To: kde-commits@kde.org
Subject: KDE/kdelibs/kdeui/colors

SVN commit 746687 by uwolfer:

proper color-picker button size

 M  +1 -1      kcolordialog.cpp  


--- trunk/KDE/kdelibs/kdeui/colors/kcolordialog.cpp #746686:746687 @@
-1154,7 +1154,7 @@
   QPushButton* button = new QPushButton( page );
   button->setIcon( KIcon("color-picker"));
   int commonHeight = addButton->sizeHint().height();
-  button->setMinimumHeight( commonHeight );
+  button->setFixedSize( commonHeight, commonHeight );
   //kDebug() << commonHeight;
   button->setIconSize(QSize(commonHeight, commonHeight));
   l_hbox->addWidget(button, 0, Qt::AlignHCenter );

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