-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/114693/
-----------------------------------------------------------

Review request for KDE Frameworks.


Repository: kio


Description
-------

If I understand correctly documentation of KFileWidget, it should be perfectly 
ok to do something like this:

KEncodingFileDialog::getOpenUrlsAndEncoding(QString(), 
QUrl("file:///etc/passwd"));

But that doesn't display the thing I'm expecting. Tracing it down I came up 
with the fix. I'm not claiming that it's the correct one, but at least in my 
situation the KFileWidget behaves as expected in all tested situations. 

Surprisingly the

void KFileWidgetPrivate::setLocationText(const QList<QUrl> &urlList)

doesn't call any setUrl, which hints that it could actually be correct.


Diffs
-----

  src/filewidgets/kfilewidget.cpp 11597b3 

Diff: https://git.reviewboard.kde.org/r/114693/diff/


Testing
-------


Thanks,

Michal Humpula

_______________________________________________
Kde-frameworks-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to