From f2bbb60419f8c5ea5992bbd8d7bdc037bf64fe3c Mon Sep 17 00:00:00 2001
From: Filip Fila <filipfila.kde@gmail.com>
Date: Fri, 5 Jul 2019 11:32:25 +0200
Subject: [PATCH] [kcmkwin] Add a clear button to the Desktop Effects search
 field

Summary: This patch adds a clear button to the Desktop Effects search field by using Kirigami's SearchField.

Test Plan: {F6973312}

Reviewers: #kwin, #vdg, davidedmundson, ngraham

Reviewed By: #kwin, #vdg, davidedmundson, ngraham

Subscribers: ngraham, kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D22286
---
 kcmkwin/kwineffects/package/contents/ui/main.qml | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/kcmkwin/kwineffects/package/contents/ui/main.qml b/kcmkwin/kwineffects/package/contents/ui/main.qml
index d0ce35a700..53568c2e6b 100644
--- a/kcmkwin/kwineffects/package/contents/ui/main.qml
+++ b/kcmkwin/kwineffects/package/contents/ui/main.qml
@@ -21,7 +21,7 @@ import QtQuick.Controls 2.5 as QtControls
 import QtQuick.Layouts 1.1
 import org.kde.kcm 1.2
 import org.kde.kconfig 1.0
-import org.kde.kirigami 2.5 as Kirigami
+import org.kde.kirigami 2.8 as Kirigami
 import org.kde.private.kcms.kwin.effects 1.0 as Private
 
 ScrollViewKCM {
@@ -36,13 +36,10 @@ ScrollViewKCM {
         }
 
         RowLayout {
-            QtControls.TextField {
+            Kirigami.SearchField {
                 id: searchField
 
                 Layout.fillWidth: true
-
-                // TODO: Needs clear button, missing in Qt.
-                placeholderText: i18n("Search...")
             }
 
             QtControls.Button {