D26530: ScrollView: Do not overlay scrollbars over contents

[Nathaniel Graham]

ngraham added a comment.


  In D26530#593152 <https://phabricator.kde.org/D26530#593152>, @mart wrote:
  
  > 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)
  
  
  Let's distinguish and lines. A frame is a four-cornered outline. What we see in the following image has no frames:
  
  F7880630: image.png <https://phabricator.kde.org/F7880630>
  
  IMO using lines to separate distinct elements that touch the bounds of the window is 1000% better than putting everything in a frame floating on the view, separated by a few pixels of padding. You can add more lines (within reason) without the view becoming overwhelming and heavy and old-fashioned looking I love love love the Kirigami style that does this, and in fact our T10891: Breeze theme evolution <https://phabricator.kde.org/T10891> task proposes moving to this style for QWidgets apps too.

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/c343fd7f/attachment.html>