diff --git a/kcmkwin/kwineffects/package/contents/ui/Effect.qml b/kcmkwin/kwineffects/package/contents/ui/Effect.qml --- a/kcmkwin/kwineffects/package/contents/ui/Effect.qml +++ b/kcmkwin/kwineffects/package/contents/ui/Effect.qml @@ -22,19 +22,10 @@ import QtQuick.Layouts 1.0 import org.kde.kirigami 2.5 as Kirigami -Rectangle { - height: row.implicitHeight - - Kirigami.Theme.inherit: false - Kirigami.Theme.colorSet: Kirigami.Theme.View - color: index % 2 ? Kirigami.Theme.backgroundColor : palette.alternateBase - - RowLayout { +Kirigami.SwipeListItem { + alternatingBackground: true + contentItem: RowLayout { id: row - - x: spacing - width: parent.width - 2 * spacing - QtControls.RadioButton { property bool _exclusive: model.ExclusiveRole != "" property bool _toggled: false @@ -130,11 +121,4 @@ onClicked: aboutItem.visible = !aboutItem.visible; } } - - // Kirigami.Theme doesn't provide alternate color. - SystemPalette { - id: palette - - colorGroup: SystemPalette.Active - } } diff --git a/kcmkwin/kwineffects/package/contents/ui/main.qml b/kcmkwin/kwineffects/package/contents/ui/main.qml --- a/kcmkwin/kwineffects/package/contents/ui/main.qml +++ b/kcmkwin/kwineffects/package/contents/ui/main.qml @@ -83,8 +83,6 @@ property var _buttonGroups: [] - spacing: Kirigami.Units.smallSpacing - model: Private.EffectsFilterProxyModel { id: searchModel @@ -97,15 +95,11 @@ } section.property: "CategoryRole" - section.delegate: Item { + section.delegate: Kirigami.SwipeListItem { width: effectsList.width - height: sectionText.implicitHeight + 2 * Kirigami.Units.smallSpacing - + sectionDelegate: true QtControls.Label { id: sectionText - - anchors.fill: parent - color: Kirigami.Theme.disabledTextColor font.weight: Font.Bold horizontalAlignment: Text.AlignHCenter