D26530: ScrollView: Do not overlay scrollbars over contents

[Arjen Hiemstra]

ahiemstra created this revision.
ahiemstra added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
ahiemstra requested review of this revision.

REVISION SUMMARY
  This ensures the scrollbar of a ScrollView do not overlap the ScrollView's
  contents. It effectively makes ScrollView's behaviour the same as all
  widget views, which also do not overlap the contents.
  
  It also removes the need for a lot of workarounds in applications and other
  places to fix the overlapping problem.

TEST PLAN
  Tested with this QML:
  
    import QtQuick 2.12
    import QtQuick.Controls 2.12
    
    import org.kde.kirigami 2.11 as Kirigami
    
    Kirigami.ApplicationWindow {
        pageStack.initialPage: Kirigami.Page {
            leftPadding: 0
            rightPadding: 0
            topPadding: 0
            bottomPadding: 0
            ScrollView {
                anchors.fill: parent
    
                ListView {
                    model: 100
    
                    delegate: ItemDelegate {
                        width: ListView.view.width
                        text: modelData
                    }
                }
            }
        }
    }

REPOSITORY
  R858 Qt Quick Controls 2: Desktop Style

BRANCH
  scrollview_scrollbar_spacing

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

AFFECTED FILES
  org.kde.desktop/ScrollView.qml

To: ahiemstra, #plasma
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, 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/20200108/ae124d95/attachment.html>