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