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

[Martin Klapetek]

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



applets/systemtray/package/contents/ui/main.qml
<https://git.reviewboard.kde.org/r/121948/#comment51250>

    I think this comment should to the xml as people/distros wanting to customize it will look only at the xml file, not here
    
    So should be explained there what the default value is


- Martin Klapetek


On Jan. 9, 2015, 4:09 p.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121948/
> -----------------------------------------------------------
> 
> (Updated Jan. 9, 2015, 4:09 p.m.)
> 
> 
> 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/20150109/d6873cee/attachment.html>