Commit 8ea57926 authored by Martin Flöser's avatar Martin Flöser
Browse files

[kcmkwin/compositing] Improve the spacing in the Effects list

Spacing needs to be specified on the ListView instead in the delegate.
This fixes incorrect vertical alignment of the buttons in each row.

In addition a proper background is set on the ScrollView.

CCBUG: 334372
REVIEW: 118260
parent 45f1060d
......@@ -27,7 +27,7 @@ import org.kde.kwin.kwincompositing 1.0
Rectangle {
id: item
width: parent.width
height: rowEffect.implicitHeight + rowEffect.spacing
height: rowEffect.implicitHeight
color: item.ListView.isCurrentItem ? effectView.backgroundActiveColor : index % 2 ? effectView.backgroundNormalColor : effectView.backgroundAlternateColor
signal changed()
property bool checked: model.EffectStatusRole
......
......@@ -125,6 +125,10 @@ Item {
highlightOnFocus: true
Layout.fillWidth: true
Layout.fillHeight: true
Rectangle {
color: effectView.backgroundNormalColor
anchors.fill: parent
}
ListView {
function exclusiveGroupForCategory(category) {
for (var i = 0; i < effectView.exclusiveGroups.length; ++i) {
......@@ -163,6 +167,7 @@ Item {
section.property: "CategoryRole"
section.delegate: sectionHeading
spacing: col.spacing
}
}
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment