From 4fd910bbfcbdf5de7e55b2c7ece7e8a1db860673 Mon Sep 17 00:00:00 2001
From: Nate Graham <nate@kde.org>
Date: Wed, 14 Oct 2020 08:15:47 -0600
Subject: [PATCH] [applets/mediacontroller] Improve bottom margin in new layout

The player buttons looked a bit too close to the footer to my eye. This
commit adds some more whitespace under them to match the whitespace
above them.
---
 .../contents/ui/ExpandedRepresentation.qml    | 22 ++++++++++---------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/applets/mediacontroller/contents/ui/ExpandedRepresentation.qml b/applets/mediacontroller/contents/ui/ExpandedRepresentation.qml
index 5a76f5640..9d4662866 100644
--- a/applets/mediacontroller/contents/ui/ExpandedRepresentation.qml
+++ b/applets/mediacontroller/contents/ui/ExpandedRepresentation.qml
@@ -414,19 +414,20 @@ PlasmaComponents3.Page {
             }
         }
 
-        Row { // Player Controls
+        RowLayout { // Player Controls
             id: playerControls
 
             property bool enabled: root.canControl
             property int controlsSize: theme.mSize(theme.defaultFont).height * 3
 
             Layout.alignment: Qt.AlignHCenter
-            spacing: units.largeSpacing
+            Layout.bottomMargin: PlasmaCore.Units.largeSpacing
+            spacing: units.smallSpacing
 
             PlasmaComponents3.ToolButton { // Previous
-                anchors.verticalCenter: parent.verticalCenter
-                width: expandedRepresentation.controlSize
-                height: width
+                Layout.alignment: Qt.AlignVCenter
+                implicitWidth: expandedRepresentation.controlSize
+                implicitHeight: implicitWidth
                 enabled: playerControls.enabled && root.canGoPrevious
                 icon.name: LayoutMirroring.enabled ? "media-skip-forward" : "media-skip-backward"
                 onClicked: {
@@ -436,17 +437,18 @@ PlasmaComponents3.Page {
             }
 
             PlasmaComponents3.ToolButton { // Pause/Play
-                width: Math.round(expandedRepresentation.controlSize * 1.5)
-                height: width
+                Layout.alignment: Qt.AlignVCenter
+                implicitWidth: Math.round(expandedRepresentation.controlSize * 1.5)
+                implicitHeight: implicitWidth
                 enabled: root.state == "playing" ? root.canPause : root.canPlay
                 icon.name: root.state == "playing" ? "media-playback-pause" : "media-playback-start"
                 onClicked: root.togglePlaying()
             }
 
             PlasmaComponents3.ToolButton { // Next
-                anchors.verticalCenter: parent.verticalCenter
-                width: expandedRepresentation.controlSize
-                height: width
+                Layout.alignment: Qt.AlignVCenter
+                implicitWidth: expandedRepresentation.controlSize
+                implicitHeight: implicitWidth
                 enabled: playerControls.enabled && root.canGoNext
                 icon.name: LayoutMirroring.enabled ? "media-skip-backward" : "media-skip-forward"
                 onClicked: {