If an Ok button is present in a KDialog, it should always be the default button (as discussed on IRC), so there is no need to set it manually. That the Ok button is currently not the default button is simply a bug in kdeui and needs to be fixed there

svn path=/trunk/KDE/kdepim/; revision=540477

chiasmuskeyselector.cpp

@@ -16,7 +16,6 @@ ChiasmusKeySelector::ChiasmusKeySelector( QWidget* parent, const QString& captio
   : KDialog( parent, caption, Ok|Cancel )
 {
   setObjectName( "chiasmusKeySelector" );
-  setDefaultButton( Ok );
   QWidget *page = new QWidget( this );
   setMainWidget(page);
 

expirypropertiesdialog.cpp

@@ -36,7 +36,6 @@ ExpiryPropertiesDialog::ExpiryPropertiesDialog( KMFolderTree* tree, KMFolder* fo
                    KDialog::Ok|KDialog::Cancel ),
       mFolder( folder )
 {
-  setDefaultButton( KDialog::Ok );
   setModal( false );
   setObjectName( "expiry_properties" );
   setAttribute( Qt::WA_DeleteOnClose );

kmpopfiltercnfrmdlg.cpp

@@ -288,7 +288,6 @@ KMPopFilterCnfrmDlg::KMPopFilterCnfrmDlg( const QList<KMPopHeaders *> & aHeaders
                                           bool aShowLaterMsgs, QWidget *aParent )
       : KDialog( aParent, i18n("POP Filter"), Ok | Help )
 {
-  setDefaultButton( Ok );
   unsigned int rulesetCount = 0;
   //mHeaders = aHeaders;
   mShowLaterMsgs = aShowLaterMsgs;

mailinglistpropertiesdialog.cpp

@@ -56,7 +56,6 @@ MailingListFolderPropertiesDialog::MailingListFolderPropertiesDialog( QWidget* p
 {
   setObjectName( "mailinglist_properties" );
   setModal( false );
-  setDefaultButton( KDialog::Ok );
   setAttribute( Qt::WA_DeleteOnClose );
   QLabel* label;
   mLastItem = 0;

newfolderdialog.cpp

@@ -64,7 +64,6 @@ NewFolderDialog::NewFolderDialog( QWidget* parent, KMFolder *folder )
 {
   setModal( false );
   setObjectName( "new_folder_dialog" );
-  setDefaultButton( KDialog::Ok );
   setAttribute( Qt::WA_DeleteOnClose );
   if ( folder ) {
     setCaption( i18n("New Subfolder of %1", folder->prettyURL() ) );

sievedebugdialog.cpp

@@ -190,7 +190,6 @@ SieveDebugDialog::SieveDebugDialog( QWidget *parent )
 :   KDialog( parent, i18n( "Sieve Diagnostics" ), KDialog::Ok ),
     mSieveJob( 0 )
 {
-    setDefaultButton( KDialog::Ok );
     // Collect all accounts
     AccountManager *am = kmkernel->acctMgr();
     assert( am );