From: Shaun Reich <shaun.reich@kdemail.net>
Date: Sun, 1 Mar 2009 21:11:12 +0000 (+0000)
Subject: Use currentIndexChanged() signal instead of activated(), the latter does not always... 
X-Git-Url: https://cloud.milkyroute.net/gitweb/dolphin.git/commitdiff_plain/9c095f56fddc86c6f916dbae69ce1c4bea2071e3

Use currentIndexChanged() signal instead of activated(), the latter does not always do what you would think.

svn path=/trunk/KDE/kdebase/apps/; revision=933889
---

diff --git a/src/settings/columnviewsettingspage.cpp b/src/settings/columnviewsettingspage.cpp
index acfba42a8..78a987df2 100644
--- a/src/settings/columnviewsettingspage.cpp
+++ b/src/settings/columnviewsettingspage.cpp
@@ -77,7 +77,7 @@ ColumnViewSettingsPage::ColumnViewSettingsPage(QWidget* parent) :
     m_textWidthBox->addItem(i18nc("@item:inlistbox Text width", "Small"));
     m_textWidthBox->addItem(i18nc("@item:inlistbox Text width", "Medium"));
     m_textWidthBox->addItem(i18nc("@item:inlistbox Text width", "Large"));
-    connect(m_textWidthBox, SIGNAL(activated(int)), this, SIGNAL(changed()));
+    connect(m_textWidthBox, SIGNAL(currentIndexChanged(int)), this, SIGNAL(changed()));
 
     QGridLayout* textGroupLayout = new QGridLayout(textGroup);
     textGroupLayout->addWidget(fontLabel, 0, 0, Qt::AlignRight);