D11411: MPRIS control: fix instance indicator with multiple app instances

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Sat Mar 17 13:39:22 UTC 2018


kossebau created this revision.
kossebau added a reviewer: KDE Connect.
kossebau requested review of this revision.

REVISION SUMMARY
  Not sure what operator+ overload has been used exactly for the int,
  in any case it does not work as intended and needs e.g. an explicit
  QString::number() invocation.
  
  Also start with number 2 for duplicated instances.

TEST PLAN
  Start multiple instances of an MPRIS player (e.g. Gwenview). Before the
  second instance would get labelled with "Name []", with this patch it is
  labelled with "Name [2]".

REPOSITORY
  R224 KDE Connect

BRANCH
  fixmultiplemprisplayernumberindicator

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

AFFECTED FILES
  plugins/mpriscontrol/mpriscontrolplugin.cpp

To: kossebau, #kde_connect
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20180317/865f39d7/attachment.html>


More information about the KDEConnect mailing list