Differential D14067

Use KCMShell import
ClosedPublic
Actions

Authored by broulik on Jul 12 2018, 8:50 AM.

Details

Reviewers
dantti
davidedmundson
Group Reviewers
Plasma
Commits
R363:54c55eb45f54: Use KCMShell import
Summary

Move ProcessRunner into full representation since it's not needed outside the dialog anymore.
Also honor KIOSK restriction for the printer KCM

Test Plan
  • KCM still opens
  • Configure printer and show queue still works

Put the following in kdeglobals:

[KDE Control Module Restrictions]
kcm_printer_manager.desktop=false

Printer kcm would no longer show in the context menu of the applet

Diff Detail

Repository
R363 Print Manager
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik created this revision.Jul 12 2018, 8:50 AM
Restricted Application added a subscriber: kde-utils-devel. · View Herald TranscriptJul 12 2018, 8:50 AM
broulik requested review of this revision.Jul 12 2018, 8:50 AM
davidedmundson accepted this revision.Aug 8 2018, 10:34 AM
This revision is now accepted and ready to land.Aug 8 2018, 10:34 AM
Closed by commit R363:54c55eb45f54: Use KCMShell import (authored by broulik). · Explain WhyAug 8 2018, 10:50 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 37622

View Options

plasmoid/package/contents/ui/PopupDialog.qml

Loading...
View Options

plasmoid/package/contents/ui/PrinterItem.qml

Loading...
View Options

plasmoid/package/contents/ui/printmanager.qml

Loading...