> On Jan. 24, 2014, 10:18 a.m., David Edmundson wrote: > > telepathy-mpris.cpp, line 264 > > <https://git.reviewboard.kde.org/r/115282/diff/1/?file=235593#file235593line264> > > > > I think this will cause problems. > > > > If you set yourself to Invisible and have a local-xmpp account which > > doesn't support invisible. When you listen to a track it will now move all > > your accounts "Away". > > > > Could you explain the problem we're trying to solve? > > James Smith wrote: > #114569 exposed an issue where the status will noticeably flicker in > ktp-plasma-applet when the status is set by the AutoAway plugin (from > Available state) and subsequent track changes in MPRIS2 change the status > without updating the away message, which is set in AutoAway and takes > precedence. The status will quickly switch back to Away from Available, > returning the Away status message instead of the MPRIS2 status message (and > associated Available state). > > So, don't even bother updating the MPRIS2 status and message.
Why does it go back to available before it goes to whatever MPRIS is set to? - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115282/#review48184 ----------------------------------------------------------- On Jan. 24, 2014, 3:43 a.m., James Smith wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115282/ > ----------------------------------------------------------- > > (Updated Jan. 24, 2014, 3:43 a.m.) > > > Review request for Telepathy and Xuetian Weng. > > > Repository: ktp-kded-module > > > Description > ------- > > Fixes state flicker when away plugin is active and track changes in a media > player. > > > Diffs > ----- > > telepathy-mpris.cpp 1c7b98c > > Diff: https://git.reviewboard.kde.org/r/115282/diff/ > > > Testing > ------- > > Compile, run. > > > Thanks, > > James Smith > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
