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

Reply via email to