mart added inline comments. INLINE COMMENTS
> davidedmundson wrote in systemtray.cpp:136 > Is this for the DBus activated case? not only, actually this case is true for each startup after the first, during normal config restore. This is the branch always executed except when a previously unknown applet is added > davidedmundson wrote in systemtray.cpp:154 > This reintroduces the NetworkManager restarting bug. > > Applet exists, is still in applets() but now isn't marked as transient as > you're not marking it as such. don't know how to properly fix it without changes in p-f tough ( i could still call destroy() in case of mutable, tough when an applet goes away for dbus reasons, its config should actually not be removed (so shouldn't be transient).. REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D2817 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: mart, #plasma Cc: davidedmundson, sebas, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts