D6102: KUrlRequester: Set NOTIFY signal to textChanged() for text property.

Elvis Angelaccio noreply at phabricator.kde.org
Sat Jun 10 12:58:52 UTC 2017


elvisangelaccio added inline comments.

INLINE COMMENTS

> kurlrequester.h:71
>      Q_PROPERTY(QString placeholderText READ placeholderText WRITE setPlaceholderText)
> -    Q_PROPERTY(QString text READ text WRITE setText)
> +    Q_PROPERTY(QString text READ text WRITE setText NOTIFY textChanged)
>      Q_PROPERTY(Qt::WindowModality fileDialogModality READ fileDialogModality WRITE setFileDialogModality)

Don't we need to add apidox with the @since tag?

REPOSITORY
  R241 KIO

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

To: arrowdodger, #frameworks
Cc: elvisangelaccio, kfunk
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170610/fe91c9a4/attachment.html>


More information about the Kde-frameworks-devel mailing list