[Folder View] Honor editable_desktop_icons KIOSK restriction
ClosedPublic

Authored by broulik on Aug 30 2018, 8:53 AM.

Details

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.

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Aug 30 2018, 8:53 AM
Restricted Application added a project: Plasma. · View Herald TranscriptAug 30 2018, 8:53 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Aug 30 2018, 8:53 AM
hein accepted this revision.Sep 5 2018, 4:42 PM
This revision is now accepted and ready to land.Sep 5 2018, 4:42 PM
This revision was automatically updated to reflect the committed changes.