[Containment Actions ContextMenu] Port to KGlobalAccel::globalShortcut()
ClosedPublic

Authored by broulik on Aug 12 2016, 8:00 AM.

Details

Summary

Implements the TODO there and removes the manual DBus code.

Test Plan

Started plasmashell, shortcuts would still show up.
Removed a shortcut, restarted plasmashell (the previous code also didn't detect changes at runtime) and no shortcut was shown (hence the value(0) instead of first() as the latter requires the list to not be empty)

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik updated this revision to Diff 5849.Aug 12 2016, 8:00 AM
broulik retitled this revision from to [Containment Actions ContextMenu] Port to KGlobalAccel::globalShortcut().
broulik updated this object.
broulik edited the test plan for this revision. (Show Details)
broulik added a reviewer: Plasma.
broulik set the repository for this revision to R120 Plasma Workspace.
Restricted Application added a project: Plasma. · View Herald TranscriptAug 12 2016, 8:00 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik updated this object.Aug 12 2016, 8:00 AM
graesslin accepted this revision.Aug 12 2016, 9:13 AM
graesslin added a reviewer: graesslin.
This revision is now accepted and ready to land.Aug 12 2016, 9:13 AM
This revision was automatically updated to reflect the committed changes.