zzag added a comment.
> Does it make sense to dump the C++ classes in libcolorcorrect? I think it will be better to keep them here for now. Once somebody needs one of them, we can move them to libcolorcorrect (bad name). INLINE COMMENTS > davidedmundson wrote in main.qml:98 > Should > > plasmoid.removeAction("configure"); > > be in an else? Could you please explain why it needs to be in an else? REPOSITORY R114 Plasma Addons BRANCH nightcolor-applet REVISION DETAIL https://phabricator.kde.org/D25947 To: zzag, #plasma, #vdg, ngraham, davidedmundson Cc: davidedmundson, ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart