From ed747d82ab1aea8c7da92463dafd1739ab4efb1f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Martin=20Gr=C3=A4=C3=9Flin?= <mgraesslin@kde.org>
Date: Tue, 10 Jun 2014 09:27:03 +0200
Subject: [PATCH] [kcmkwin/compositing] Use Effect name as configuration window
 title

REVIEW: 118632
---
 kcmkwin/kwincompositing/effectconfig.cpp | 3 ++-
 kcmkwin/kwincompositing/effectconfig.h   | 2 +-
 kcmkwin/kwincompositing/qml/Effect.qml   | 2 +-
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/kcmkwin/kwincompositing/effectconfig.cpp b/kcmkwin/kwincompositing/effectconfig.cpp
index 1bab11a722..5ea433596e 100644
--- a/kcmkwin/kwincompositing/effectconfig.cpp
+++ b/kcmkwin/kwincompositing/effectconfig.cpp
@@ -43,10 +43,11 @@ EffectConfig::EffectConfig(QObject *parent)
 {
 }
 
-void EffectConfig::openConfig(const QString &serviceName, bool scripted)
+void EffectConfig::openConfig(const QString &serviceName, bool scripted, const QString &title)
 {
     //setup the UI
     QDialog dialog;
+    dialog.setWindowTitle(title);
 
     // create the KCModule through the plugintrader
     KCModule *kcm = nullptr;
diff --git a/kcmkwin/kwincompositing/effectconfig.h b/kcmkwin/kwincompositing/effectconfig.h
index 09967e8ca4..73e20d124c 100644
--- a/kcmkwin/kwincompositing/effectconfig.h
+++ b/kcmkwin/kwincompositing/effectconfig.h
@@ -36,7 +36,7 @@ public:
     explicit EffectConfig(QObject *parent = 0);
     QString serviceName(const QString &serviceName);
 
-    Q_INVOKABLE void openConfig(const QString &effectName, bool scripted);
+    Q_INVOKABLE void openConfig(const QString &effectName, bool scripted, const QString &title);
     Q_INVOKABLE void openGHNS();
 
 Q_SIGNALS:
diff --git a/kcmkwin/kwincompositing/qml/Effect.qml b/kcmkwin/kwincompositing/qml/Effect.qml
index 5b11a41502..e05ab81cef 100644
--- a/kcmkwin/kwincompositing/qml/Effect.qml
+++ b/kcmkwin/kwincompositing/qml/Effect.qml
@@ -143,7 +143,7 @@ Rectangle {
             enabled: effectStatusCheckBox.checked
             iconName: "configure"
             onClicked: {
-                effectConfig.openConfig(model.ServiceNameRole, model.ScriptedRole);
+                effectConfig.openConfig(model.ServiceNameRole, model.ScriptedRole, model.NameRole);
             }
         }