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

Gleb Popov noreply at phabricator.kde.org
Mon Jun 5 14:18:37 UTC 2017


arrowdodger created this revision.
Restricted Application added a project: Frameworks.

REVISION SUMMARY
  Having `NOTIFY textChanged` on `text` property may come useful when using KUrlRequester with KConfig. KUrlRequester field may hold paths for executables and sometimes it is more convenient to specify only executable name, so that it would be looked in PATH. But KConfig uses `setUrl()` method for saving KUrlRequester values, which turns `foo` into `/absolute/path/to/current/working/dir/foo`, which is obviouslt wrong.

REPOSITORY
  R241 KIO

BRANCH
  master

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

AFFECTED FILES
  src/widgets/kurlrequester.h

To: arrowdodger, #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170605/24aaafd9/attachment.html>


More information about the Kde-frameworks-devel mailing list