From 89126c7323a6764cbe3c961953e3361f4581b641 Mon Sep 17 00:00:00 2001
From: Ghostkeeper <rubend@tutanota.com>
Date: Tue, 27 Nov 2018 13:08:41 +0100
Subject: [PATCH] Use background instead of contentItem

This way the padding and such doesn't matter any more. And the original background disappears, which was unthemed and still behind the original contentItem.

Contributes to issue CURA-5876.
---
 resources/qml/Menus/ConfigurationMenu/ConfigurationItem.qml | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/resources/qml/Menus/ConfigurationMenu/ConfigurationItem.qml b/resources/qml/Menus/ConfigurationMenu/ConfigurationItem.qml
index 2988ab9eec..297c95d1d1 100644
--- a/resources/qml/Menus/ConfigurationMenu/ConfigurationItem.qml
+++ b/resources/qml/Menus/ConfigurationMenu/ConfigurationItem.qml
@@ -15,11 +15,8 @@ Button
     signal activateConfiguration()
 
     height: childrenRect.height
-    padding: 0 //Stupid QML button has spacing by default.
-    rightPadding: 0
-    leftPadding: 0
 
-    contentItem: Rectangle
+    background: Rectangle
     {
         height: childrenRect.height
         color: parent.hovered ? UM.Theme.getColor("action_button_hovered") : UM.Theme.getColor("action_button")