D15331: [Folder View] Hide /home/foo/Desktop place in configuration

[Kai Uwe Broulik]

broulik created this revision.
broulik added reviewers: Plasma, hein, fvogt.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  Folder View is meant to use `desktop:/` and having two entries that behave differently can generate confusion.
  
  CCBUG: 398141

TEST PLAN
  There is no "Desktop" entry in places by default but I have seen many users add it
  
  Existing configuration will instead show the path as custom url in the configuration

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D15331

AFFECTED FILES
  containments/desktop/package/contents/ui/ConfigLocation.qml
  containments/desktop/plugins/folder/placesmodel.cpp
  containments/desktop/plugins/folder/placesmodel.h

To: broulik, #plasma, hein, fvogt
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180907/663b9289/attachment.html>