From a03e1be6011362cb044cc589373640e60aab1e5a Mon Sep 17 00:00:00 2001
From: Ghostkeeper <rubend@tutanota.com>
Date: Mon, 26 Nov 2018 17:19:32 +0100
Subject: [PATCH] Simplify sectionHeading element

No need to define that component in a higher location, just define it where you need it. Also, no need to use a Rectangle for this if it has no colour, just use an Item, but the Item itself is also not necessary if you just need the padding.

Contributes to issue CURA-5876.
---
 .../ConfigurationListView.qml                 | 23 ++++++-------------
 1 file changed, 7 insertions(+), 16 deletions(-)

diff --git a/resources/qml/Menus/ConfigurationMenu/ConfigurationListView.qml b/resources/qml/Menus/ConfigurationMenu/ConfigurationListView.qml
index be9aad6d04..25dc3fac11 100644
--- a/resources/qml/Menus/ConfigurationMenu/ConfigurationListView.qml
+++ b/resources/qml/Menus/ConfigurationMenu/ConfigurationListView.qml
@@ -25,21 +25,6 @@ Column
         }
     }
 
-    Component
-    {
-        id: sectionHeading
-        Rectangle
-        {
-            height: childrenRect.height + UM.Theme.getSize("default_margin").height
-            Label
-            {
-                text: section
-                font: UM.Theme.getFont("default_bold")
-                color: UM.Theme.getColor("configuration_item_text")
-            }
-        }
-    }
-
     ScrollView
     {
         id: container
@@ -58,7 +43,13 @@ Column
 
             section.property: "modelData.printerType"
             section.criteria: ViewSection.FullString
-            section.delegate: sectionHeading
+            section.delegate: Label
+            {
+                text: section
+                font: UM.Theme.getFont("default_bold")
+                color: UM.Theme.getColor("configuration_item_text")
+                bottomPadding: UM.Theme.getSize("default_margin").height
+            }
 
             model: (outputDevice != null) ? outputDevice.uniqueConfigurations : []
             delegate: ConfigurationItem