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
