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

List:       kde-commits
Subject:    [kde-baseapps/KDE/4.9] dolphin/src/panels/information: Do not use a hardcoded size for the buttons i
From:       Frank Reininghaus <frank78ac () googlemail ! com>
Date:       2012-11-04 13:41:20
Message-ID: 20121104134120.CEB58A6091 () git ! kde ! org
[Download RAW message or body]

Git commit cc06935397ecb1c543ed7d52887007f425a1cddb by Frank Reininghaus.
Committed on 04/11/2012 at 14:39.
Pushed by freininghaus into branch 'KDE/4.9'.

Do not use a hardcoded size for the buttons in the Information panel

Thanks to Kai Uwe Broulok and Jekyll Wu for the investigation!

BUG: 309498
FIXED-IN: 4.9.4

M  +5    -2    dolphin/src/panels/information/phononwidget.cpp

http://commits.kde.org/kde-baseapps/cc06935397ecb1c543ed7d52887007f425a1cddb

diff --git a/dolphin/src/panels/information/phononwidget.cpp \
b/dolphin/src/panels/information/phononwidget.cpp index f0c030e..a36ada1 100644
--- a/dolphin/src/panels/information/phononwidget.cpp
+++ b/dolphin/src/panels/information/phononwidget.cpp
@@ -123,14 +123,17 @@ void PhononWidget::showEvent(QShowEvent *event)
 
         m_topLayout->addLayout(controlsLayout);
 
+        const int smallIconSize = IconSize(KIconLoader::Small);
+        const QSize buttonSize(smallIconSize, smallIconSize);
+
         m_playButton->setToolTip(i18n("play"));
-        m_playButton->setIconSize(QSize(16, 16));
+        m_playButton->setIconSize(buttonSize);
         m_playButton->setIcon(KIcon("media-playback-start"));
         m_playButton->setAutoRaise(true);
         connect(m_playButton, SIGNAL(clicked()), this, SLOT(play()));
 
         m_stopButton->setToolTip(i18n("stop"));
-        m_stopButton->setIconSize(QSize(16, 16));
+        m_stopButton->setIconSize(buttonSize);
         m_stopButton->setIcon(KIcon("media-playback-stop"));
         m_stopButton->setAutoRaise(true);
         m_stopButton->hide();


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