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

Sat, 10 Jun 2017 05:59:19 -0700 

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

Reply via email to