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

Reply via email to