Review Request 123789: automatically add newly installed plasmoids in the systray

Marco Martin notmart at gmail.com
Thu May 14 15:17:59 UTC 2015


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

(Updated May 14, 2015, 3:17 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Changes
-------

Submitted with commit b494fe13770f25058179f93ab7cf355517078166 by Marco Martin to branch master.


Bugs: 345872
    https://bugs.kde.org/show_bug.cgi?id=345872


Repository: plasma-workspace


Description
-------

if new plasmoids are installed that are systray enabled and enabled by default, automatically add them, unless they were disabled explicitly. This solves a problem distributions seem to add while updating to new plasma releases.

obtain this by keeping both a list of explicitly enabled stuff and all the already known stuff

it does horrible things with arrays, that's due to some weird sideeffects of having qstringlists as properties, of which we'll have to pay close attention to


Diffs
-----

  applets/systemtray/package/contents/config/main.xml 2967dba 
  applets/systemtray/package/contents/ui/main.qml fd59a14 

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


Testing
-------


Thanks,

Marco Martin

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


More information about the Plasma-devel mailing list