ngraham added a comment.

  Whether or not it's better is not relevant for this patch, which right now is 
only about fixing the text color. It's best not to mix multiple changes like 
this, unless you want to expand the patch in scope, in which case it should be 
re-titled accordingly.
  
  Still, I would recommend doing one change per patch, because it makes each 
one more likely to get in compared to patches with multiple changes (especially 
when any one of those individual changes may be controversial or require 
discussion).

REPOSITORY
  R98 Breeze for Gtk

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

To: mthw, #vdg, #breeze, ndavis, ngraham
Cc: ndavis, zzag, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, 
ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart

Reply via email to