ngraham added inline comments. INLINE COMMENTS
> broulik wrote in kfileplacesmodel.cpp:245 > Desktop KIO claims that this may return $HOME if the Desktop folder does not > exist, can you confirm/invalidate that? If I delete ~/Desktop, trying to access it via the Places entry gives me a red KMessageWidget that says "The file or folder /home/test/Desktop does not exist." Still, that folder exists by default, and I think the kind of user who deletes their Desktop folder is the kind of user we're not targeting with this change. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D11768 To: ngraham, #dolphin, #gwenview, #frameworks, #vdg Cc: broulik, michaelh, ngraham