From 0417f736d3f5d7afc08a221e329d1ce2af542e51 Mon Sep 17 00:00:00 2001
From: Nate Graham <nate@kde.org>
Date: Thu, 7 Jan 2021 19:55:16 -0700
Subject: [PATCH] [wallpapers/image] Tweak slideshow folders list appearance

This makes it look better with the change to Kirigami.SwipeListItem to
make the actions always visible on the desktop, and in the process fixes
a binding loop and some minor visual glitches.
---
 .../image/imagepackage/contents/ui/config.qml     | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/wallpapers/image/imagepackage/contents/ui/config.qml b/wallpapers/image/imagepackage/contents/ui/config.qml
index e5c9067f9..fe30e3030 100644
--- a/wallpapers/image/imagepackage/contents/ui/config.qml
+++ b/wallpapers/image/imagepackage/contents/ui/config.qml
@@ -306,22 +306,21 @@ ColumnLayout {
                 QtControls2.ScrollView {
                     id: foldersScroll
                     Layout.fillHeight: true
-                    Layout.preferredWidth: 0.25 * parent.width
+                    Layout.preferredWidth: 0.35 * parent.width
+                    Layout.maximumWidth: Kirigami.Units.gridUnit * 16
                     Component.onCompleted: foldersScroll.background.visible = true;
                     ListView {
                         id: slidePathsView
                         model: imageWallpaper.slidePaths
                         delegate: Kirigami.SwipeListItem {
-                            id: folderDelegate
                             width: slidePathsView.width
+                            // Don't need a highlight
+                            activeBackgroundColor: "transparent"
                             contentItem: Kirigami.BasicListItem {
-                                // The parent item already has a highlight
+                                // Don't need a highlight
                                 activeBackgroundColor: "transparent"
-                                // Otherwise there are unnecessary margins
-                                anchors.top: parent.top
-                                anchors.bottom: parent.bottom
-                                anchors.left: parent.left
-                                // No right anchor so text can be elided by actions
+                                activeTextColor: "black"
+                                separatorVisible: false
 
                                 // Header: the folder
                                 label: {