From 5f1e8ec2bd7537f9a4cf8c00c5e21911ec3bc60a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Martin=20Gr=C3=A4=C3=9Flin?= <mgraesslin@kde.org>
Date: Fri, 13 Dec 2013 12:24:18 +0100
Subject: [PATCH] Use RowLayout for the video, configure and about buttons

Simplifies the code as we don't have to anchor the buttons.

REVIEW: 114432
---
 kcmkwin/kwincompositing/qml/Effect.qml | 50 +++++++++++++-------------
 1 file changed, 25 insertions(+), 25 deletions(-)

diff --git a/kcmkwin/kwincompositing/qml/Effect.qml b/kcmkwin/kwincompositing/qml/Effect.qml
index 20ab18c9e9..32a9e67815 100644
--- a/kcmkwin/kwincompositing/qml/Effect.qml
+++ b/kcmkwin/kwincompositing/qml/Effect.qml
@@ -182,32 +182,32 @@ Item {
                 }
             }
 
-            Button {
-                id: videoButton
-                anchors.right: effectConfig.effectUiConfigExists(model.ServiceNameRole) ? configureButton.left : aboutButton.left
-                visible: model.VideoRole.toString() !== ""
-                iconName: "video"
-                onClicked: videoItem.showHide()
-            }
-            Button {
-                id: configureButton
-                anchors.right: aboutButton.left
-                visible: effectConfig.effectUiConfigExists(model.ServiceNameRole)
-                enabled: effectStatusCheckBox.checked
-                iconName: "configure"
-                onClicked: {
-                    effectConfig.openConfig(model.NameRole);
-                }
-            }
-
-            Button {
-                id: aboutButton
+            RowLayout {
                 anchors.right: parent.right
-                iconName: "dialog-information"
-                onClicked: {
-                    animationAbout.running = true;
-                    animationAboutSpacing.running = true;
-                    desc.wrapDescription();
+                Button {
+                    id: videoButton
+                    visible: model.VideoRole.toString() !== ""
+                    iconName: "video"
+                    onClicked: videoItem.showHide()
+                }
+                Button {
+                    id: configureButton
+                    visible: effectConfig.effectUiConfigExists(model.ServiceNameRole)
+                    enabled: effectStatusCheckBox.checked
+                    iconName: "configure"
+                    onClicked: {
+                        effectConfig.openConfig(model.NameRole);
+                    }
+                }
+
+                Button {
+                    id: aboutButton
+                    iconName: "dialog-information"
+                    onClicked: {
+                        animationAbout.running = true;
+                        animationAboutSpacing.running = true;
+                        desc.wrapDescription();
+                    }
                 }
             }