ngraham created this revision. ngraham added reviewers: VDG, Breeze. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. ngraham requested review of this revision.
REVISION SUMMARY This extra padding was no big deal when using the current Breeze style, where scrollable views float amorphously over the background with no clearly-defined edges, but becomes highly noticeable in a bad way when moving towards enclosing views between single-pixel lines, as for T11661 <https://phabricator.kde.org/T11661>. In this case, you want for the scrollview to touch the line. TEST PLAN (With D27669 <https://phabricator.kde.org/D27669> also applied) Dolphin's Places panel, before: F8133813: Before.png <https://phabricator.kde.org/F8133813> After: F8133812: After.png <https://phabricator.kde.org/F8133812> Doesn't look any worse without D27669 <https://phabricator.kde.org/D27669> applied either. REPOSITORY R31 Breeze BRANCH master REVISION DETAIL https://phabricator.kde.org/D27685 AFFECTED FILES kstyle/breezestyle.cpp To: ngraham, #vdg, #breeze Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart