D11021: [Media controller] Add simple volume control

Radek HuĊĦek noreply at phabricator.kde.org
Fri Mar 30 19:06:59 UTC 2018


Pitel updated this revision to Diff 30963.
Pitel added a comment.


    Convert `changeVolume` helper into a Job... Right now it does not check wheter the DBus call is successful - I tried to wire it in but ended with segfault. I will investigate it further. Except that I hope it is ok.
    
    (I also started thinking how to change the job system into something easier to work with. Were there any attempts in this direction?)

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D11021?vs=28729&id=30963

BRANCH
  mediacontroller

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

AFFECTED FILES
  applets/mediacontroller/contents/ui/main.qml
  dataengines/mpris2/mpris2.operations
  dataengines/mpris2/multiplexedservice.cpp
  dataengines/mpris2/playeractionjob.cpp
  dataengines/mpris2/playercontrol.cpp
  dataengines/mpris2/playercontrol.h

To: Pitel, #plasma, broulik
Cc: broulik, nicolasfella, plasma-devel, ragreen, 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/20180330/58c7db45/attachment-0001.html>


More information about the Plasma-devel mailing list