[Differential] [Commented On] D3071: Port away from KDELibs4Support

apol (Aleix Pol Gonzalez) noreply at phabricator.kde.org
Sun Oct 23 15:04:37 UTC 2016


apol added a comment.


  Other than that, LGTM.

INLINE COMMENTS

> passworddialog.cpp:65
>      connect(verifyEdit, SIGNAL(textEdited(QString)), SLOT(passwordChanged()));
> +    connect(buttons->button(QDialogButtonBox::Ok), SIGNAL(clicked(bool)), this, SLOT(accept()));
> +    connect(buttons->button(QDialogButtonBox::Cancel), SIGNAL(clicked(bool)), this, SLOT(reject()));

It's better to use the dialog box directly, rather than accessing the buttons:

  connect(buttonBox, &QDialogButtonBox::accepted, dialog, &QDialog::accept);
  connect(buttonBox, &QDialogButtonBox::rejected, dialog, &QDialog::reject);

REPOSITORY
  rUSERMANAGER User Manager

REVISION DETAIL
  https://phabricator.kde.org/D3071

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidk, #plasma, mart
Cc: apol, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161023/38dccf5e/attachment.html>


More information about the Plasma-devel mailing list