hpereiradacosta added a comment.

  >> my complain here is that this argument was either not made or discarded, 
when the first switch to thin scrollbar was done.
  >>  This is my main concern about this change: the going back and forth using 
adhoc arguments each time to justify the change, often contradicting each 
other.  It means essentially that we either don't know what we are doing, or 
dont think our changes enough. This is bad imo
  > 
  > If I had been around back then or noticed the patch, I would have made this 
argument.
  
  For the record, you were around for the original commit that made the 
scrollbar thiner (https://phabricator.kde.org/D9792)

REPOSITORY
  R31 Breeze

BRANCH
  phab/scrollbarseparator

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

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

Reply via email to