D15160: [Folder View] Honor editable_desktop_icons KIOSK restriction

Kai Uwe Broulik noreply at phabricator.kde.org
Thu Aug 30 09:53:21 BST 2018


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

REVISION SUMMARY
  Disallows moving desktop icons, basically force-enables the "positions locked" option

TEST PLAN
  - Verified that the checkbox in settings and the context menu was checked and disabled
  - Verified that it didn't touch the stored config so disabling the restriction would restore movable icons if it was enabled before
  - Verified that changing the setting works if restriction isnt enabled
  
  The docs also mention "renaming" and "removing" icons but this should be done by making Desktop folder read-only, which FolderView also doesn't handle very well. Dolphin, for instance, disables renaming and deleting right away whereas Folder View allows and then fails when doing so. Will fix that separately.

REPOSITORY
  R119 Plasma Desktop

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

AFFECTED FILES
  containments/desktop/package/contents/ui/ConfigIcons.qml
  containments/desktop/package/contents/ui/FolderViewLayer.qml
  containments/desktop/plugins/folder/viewpropertiesmenu.cpp
  containments/desktop/plugins/folder/viewpropertiesmenu.h

To: broulik, #plasma, davidedmundson
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/20180830/013d527a/attachment.html>


More information about the Plasma-devel mailing list