Review Request 121948: When initializing plasma, add to the system tray the plasmoids marked as EnabledByDefault

[Aleix Pol Gonzalez]

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

(Updated Jan. 10, 2015, 1:15 a.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


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


Repository: plasma-workspace


Description
-------

So far, we used to have a hardcoded list of plasmoids to have over there. While this works this can have some drawbacks (see attached bug). In this case, it's not possible for muon to appear over there by default (which makes complete sense) and it's not acceptable to get muon on the default list because well, it might not be there.

Another very similar use-case is KTp presence plasmoid. It can't be hardcoded as it's not even part of plasma, but if you have KTp installed you want it there, because it's needed for a fluent usage of the suite.


Diffs
-----

  applets/systemtray/package/contents/config/main.xml 5f8c598 
  applets/systemtray/package/contents/ui/main.qml 36e31d0 
  applets/systemtray/plugin/host.h e8db75c 
  applets/systemtray/plugin/host.cpp 117c2a0 

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


Testing
-------

Removed my config, ran it again, the plasmoids marked as "X-KDE-PluginInfo-EnabledByDefault=true" and NotificationArea-proof appeared.


Thanks,

Aleix Pol Gonzalez

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