broulik accepted this revision.
broulik added a comment.
This revision is now accepted and ready to land.


  Good idea. I'm somewhat concerned by the fact that media controller applet 
only loads when you have an mpris service running and then we have applications 
like Amarok that also try to register them which might do so before we do this 
here? Certainly better approach than my original patch.

INLINE COMMENTS

> multiplexedservice.h:43
> +public Q_SLOTS:
> +    void enableGlobalShortcuts();
> +

Doesn't this need to be registered in some .services file? Or is that just 
needed for "startOperationCall"?

REPOSITORY
  R120 Plasma Workspace

BRANCH
  master

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

To: davidedmundson, #plasma, broulik
Cc: broulik, plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

Reply via email to