dfaure added a comment.
Agreed about notification delegate, will do. Does this really use KServiceAction anywhere? I don't see that. Confusing this one and D28988 <https://phabricator.kde.org/D28988>? [which however can't use the KServiceAction because of special support for %D and stuff] REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D28990 To: dfaure, drosca, davidedmundson, broulik Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart