From 505ce9929b2f36d8e29330f0accfbb83d654a8cd Mon Sep 17 00:00:00 2001
From: David Edmundson <kde@davidedmundson.co.uk>
Date: Wed, 15 Jan 2020 10:43:58 +0000
Subject: [PATCH] [sddm-theme] Don't have a broken reveal password button

Summary:
sddm-greeter will have a button for the reveal password button, but due
to sddm-greeter not loading a relevant QPT has no code to force it to
load the breeze icon set.

Without the breeze icon set, the clear button does not show.

There are ways to solve this, but none are trivial or reliable.

I threatened to do a revert in 5.12 (https://phabricator.kde.org/D9040)
but the bug has still not been fixed since.

BUG: 396039
CCBUG: 387418

Reviewers: #plasma

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D26675
---
 sddm-theme/Login.qml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sddm-theme/Login.qml b/sddm-theme/Login.qml
index 0e973a94b..49d6805d8 100644
--- a/sddm-theme/Login.qml
+++ b/sddm-theme/Login.qml
@@ -75,7 +75,7 @@ SessionManagementScreen {
             placeholderText: i18nd("plasma_lookandfeel_org.kde.lookandfeel", "Password")
             focus: !showUsernamePrompt || lastUserName
             echoMode: TextInput.Password
-            revealPasswordButtonShown: true
+            revealPasswordButtonShown: false // Disabled whilst SDDM does not have the breeze icon set loaded
 
             onAccepted: {
                 if (root.loginScreenUiVisible) {