Review Request 119430: Don't remove dbus-activated plasmoids too early.

Anton Kreuzkamp akreuzkamp at web.de
Wed Jul 23 16:54:37 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119430/
-----------------------------------------------------------

Review request for Plasma and Sebastian Kügler.


Repository: plasma-workspace


Description
-------

The systemtray plugin for dbus-activated plasmoids removed the
plasmoid as soon as a service vanished, that matched the pattern.
This behavior broke as soon as there're multiple matchin services,
e.g. the mediacontroller plasmoid when multiple mpris2-mediaplayers
are started. As soon as you quit one application, the plasmoid would
get removed.

With this patch the plasmoid only gets removed after the last service
quit.


Diffs
-----

  applets/systemtray/plugin/protocols/plasmoid/plasmoidprotocol.h e77bae1f58377068be7c306d56073b626240abcd 
  applets/systemtray/plugin/protocols/plasmoid/plasmoidprotocol.cpp 992c7f3226cc0f71c085e9148179b2e44dcd2e4f 

Diff: https://git.reviewboard.kde.org/r/119430/diff/


Testing
-------

Only tested with mediacontroller plasmoid. Works there.


Thanks,

Anton Kreuzkamp

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140723/e87525c4/attachment.html>


More information about the Plasma-devel mailing list