D26530: ScrollView: Do not overlay scrollbars over contents

[Arjen Hiemstra]

ahiemstra added a comment.


  > yes, i'm saying that the patch besides putting the separation in the frame, it also reintroduces the frame by default, which is unrelated (and i think it should be avoided)
  
  The patch as-is does not, what you see is the default scrollview background that I enabled in my tiny test. This patch actually just makes the space for the scrollbar, since the scrollbar itself was not changed whatever is underneath becomes visible. With the default scroll view background enabled that becomes the frame, but it can easily just be a view-colored rect.

REPOSITORY
  R858 Qt Quick Controls 2: Desktop Style

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

To: ahiemstra, #plasma, #goal_consistency, mart
Cc: kmaterka, mart, ndavis, IlyaBizyaev, ngraham, broulik, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200113/b9ad06ed/attachment.html>