D26385: Kicker/RecentDocuments Avoid referring to hidden places

[Méven Car]

meven added a comment.


  In D26385#587173 <https://phabricator.kde.org/D26385#587173>, @hein wrote:
  
  > I don't agree with this patch. Hiding a place or section from the Places panel doesn't mean the user didn't add it and it's not to be used. We don't hide it from Folder View's config or relative title generator either. Hidden is not removed. Please revert.
  
  
  I meant to do this, as the use might be confused to discover a place name he never sees.
  For instance "Hard disk (400 GB)" (mountpoint /) might be hidden but will be the prefix to all local files not in other places.
  To me it is confusing, but for consistency let's keep those visible.
  
  Reverted in 4446d687d337dc3f33fc50ca65a8beb161367204 <https://phabricator.kde.org/R120:4446d687d337dc3f33fc50ca65a8beb161367204>

REPOSITORY
  R120 Plasma Workspace

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

To: meven, #plasma, ngraham, hein
Cc: hein, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200103/82d7ed2c/attachment-0001.html>