Hi,
In qt4 it was a KDialog so okClicked() and applyClicked() existed.
Now it’s a KPageDialog => a QDialog so we must to connect to QDialogButtonBox.

This patch fixes bad signal.

Can I commit it ?

Thanks


-- 
Laurent Montel | laurent.mon...@kdab.com | KDE/Qt Senior Software Engineer
KDAB (France) S.A.S., a KDAB Group company
Tel. France +33 (0)4 90 84 08 53,  http://www.kdab.fr

diff --git a/src/ksettings/dialog.cpp b/src/ksettings/dialog.cpp
index bce70cb..96f8dcb 100644
--- a/src/ksettings/dialog.cpp
+++ b/src/ksettings/dialog.cpp
@@ -430,8 +430,8 @@ void DialogPrivate::createDialogFromServices()
     // I have no idea how to check that in KPluginSelector::load()...
     //q->showButton(KDialog::User1, true);
 
-    QObject::connect(q, SIGNAL(okClicked()), q, SLOT(_k_syncConfiguration()));
-    QObject::connect(q, SIGNAL(applyClicked()), q, SLOT(_k_syncConfiguration()));
+    QObject::connect(q->button(QDialogButtonBox::Ok), SIGNAL(clicked()), q, SLOT(_k_syncConfiguration()));
+    QObject::connect(q->button(QDialogButtonBox::Apply), SIGNAL(clicked()), q, SLOT(_k_syncConfiguration()));
     QObject::connect(q, SIGNAL(configCommitted(QByteArray)), q,
                      SLOT(_k_reparseConfiguration(QByteArray)));
 }
_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to