D15739: [Places panel] Don't show Root by default

Nathaniel Graham noreply at phabricator.kde.org
Tue Sep 25 03:27:49 BST 2018


ngraham created this revision.
ngraham added reviewers: Dolphin, VDG.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
ngraham requested review of this revision.

REVISION SUMMARY
  This patch removes Root from the default Places panel for new installations, for the following reasons:
  
  - It's redundant; you can still get to / on your machine in one click using the appropriate disk entry on the bottom of the Places panel.
  - The Root item shows up in red, indicating its danger. Dangerous items are probably not a good idea to show by default, especially not right next to safe items such as Home.
  - It's not very useful. On Linux systems, messing around with / is discouraged, and Dolphin doesn't even yet have the ability to modify files and folders there anyway even if the user wanted to; it's purely read-only. The usefulness of such a thing is questionable.
  - By removing it, we gain room to add something more useful such as a Recent Documents item (D7446 <https://phabricator.kde.org/D7446>) without making the Places panel show a vertical scrollbar with Dolphin's default size.

TEST PLAN
  Login in with a new user account or rm `~/.local/share/user-places.xbel` and open Dolphin; notice that Root is not on the Places panel.
  
  [image goes here]
  
  Existing user accounts are untouched.

REPOSITORY
  R241 KIO

BRANCH
  remote-root-from-places-panel (branched from master)

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

AFFECTED FILES
  src/filewidgets/kfileplacesmodel.cpp

To: ngraham, #dolphin, #vdg
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180925/c9e6f17a/attachment.html>


More information about the Kde-frameworks-devel mailing list