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