[Differential] [Updated, 159 lines] D4157: [Icon Applet] Use KIO::statJob to work with remote URLs
Kai Uwe Broulik
noreply at phabricator.kde.org
Mon Jan 16 18:38:57 UTC 2017
broulik updated this revision to Diff 10253.
broulik added a comment.
- Don't bail out just because stat job failed, otherwise when user manually changes Link to a non-existing file it locks him out of the icon applet so he can never correct this mistake
- Clean up a bit
It will now always show file extension (used to do QFileInfo(localUrlString).baseName() before)
REPOSITORY
R120 Plasma Workspace
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D4157?vs=10247&id=10253
REVISION DETAIL
https://phabricator.kde.org/D4157
AFFECTED FILES
applets/icon/iconapplet.cpp
applets/icon/package/contents/ui/main.qml
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: broulik, #plasma, mart, dfaure
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170116/0d5d5772/attachment.html>
More information about the Plasma-devel
mailing list