D27669: [kstyle] Tools area

[David Redondo]

davidre added a comment.


  I removed the separator line for comparison:
  F8168553: grafik.png <https://phabricator.kde.org/F8168553>
  F8168557: Screenshot_20200310_095331.PNG <https://phabricator.kde.org/F8168557>
  I thought the goal was to have less separators (see the attached task and this effectively increases the count. KDevelop before, your patch and your patch without separator (ignore the broken toolbar) :F8168568: Screenshot_20200310_095946.PNG <https://phabricator.kde.org/F8168568>F8168562: Screenshot_20200310_095741.PNG <https://phabricator.kde.org/F8168562>
  
  F8168570: Screenshot_20200310_100102.PNG <https://phabricator.kde.org/F8168570>
  All in all I think this patch would also work without the separator line

REPOSITORY
  R31 Breeze

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

To: cblack, #plasma, #breeze, #vdg
Cc: davidre, davidedmundson, hpereiradacosta, gvgeo, ngraham, manueljlin, niccolove, ndavis, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200310/5c2b0de6/attachment-0001.html>