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

Reply via email to