From d0e0f6c88cdd032d3bdb9dc7b0c85902fb3d859a Mon Sep 17 00:00:00 2001
From: David Edmundson <kde@davidedmundson.co.uk>
Date: Mon, 28 Nov 2016 07:49:41 +0000
Subject: [PATCH] Align configure button consistently across delegates

Summary:
Currently the configure button is aligned to the win decoration in the
background. This is bound to 40 - the inactive decorations shadow.

This means the configure button ends up in different places across each
item,
looking a bit weird and potentially not even on top of the right
delegate.

This patch aligns the configure button relative to the overall delegate
so that it's in the same place on every item

Test Plan: Looked at KCM

Reviewers: #plasma

Subscribers: plasma-devel, kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D3526

BUG: 372685
---
 kcmkwin/kwindecoration/qml/Previews.qml | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/kcmkwin/kwindecoration/qml/Previews.qml b/kcmkwin/kwindecoration/qml/Previews.qml
index fe4c5b3274..be8036c323 100644
--- a/kcmkwin/kwindecoration/qml/Previews.qml
+++ b/kcmkwin/kwindecoration/qml/Previews.qml
@@ -107,8 +107,9 @@ ScrollView {
                     Button {
                         id: configureButton
                         anchors {
-                            left: inactivePreview.left
-                            bottom: inactivePreview.bottom
+                            left: parent.left
+                            bottom: parent.bottom
+                            margins: 20
                         }
                         enabled: model["configureable"]
                         iconName: "configure"