ngraham added a comment.

  Very nice, I applaud the goal. However it looks like on net a lot more lines 
of code were added than were removed, which suggests the addition of redundant 
boilerplate or unrelated style refactoring. In the former case, can we factor 
out some of it into additional helper functions?

REPOSITORY
  R31 Breeze

REVISION DETAIL
  https://phabricator.kde.org/D25543

To: ndavis, #plasma, #breeze
Cc: ngraham, IlyaBizyaev, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, 
fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to