Commit b3030730 authored by Nate Graham's avatar Nate Graham
Browse files

[Lock screen] Make clearPassword() do what it says and then use it

Right now the clearPassword() signal does not actually clear the
password; it only selects all text. This is a violation of its name
as well as being pointless since there's no longer a way to see the
unmasked text, which means you always need to re-enter the whole
password anyway.

BUG: 412252
FIXED-IN: 5.20
parent 31eb253b
...@@ -64,7 +64,7 @@ PlasmaCore.ColorScope { ...@@ -64,7 +64,7 @@ PlasmaCore.ColorScope {
Connections { Connections {
target: sessionManagement target: sessionManagement
onAboutToSuspend: { onAboutToSuspend: {
mainBlock.mainPasswordBox.text = ""; root.clearPassword();
} }
} }
...@@ -121,7 +121,7 @@ PlasmaCore.ColorScope { ...@@ -121,7 +121,7 @@ PlasmaCore.ColorScope {
inputPanel.showHide(); inputPanel.showHide();
} }
if (!uiVisible) { if (!uiVisible) {
mainBlock.mainPasswordBox.text = ""; root.clearPassword();
} }
} }
Keys.onPressed: { Keys.onPressed: {
......
...@@ -89,7 +89,7 @@ SessionManagementScreen { ...@@ -89,7 +89,7 @@ SessionManagementScreen {
target: root target: root
onClearPassword: { onClearPassword: {
passwordBox.forceActiveFocus() passwordBox.forceActiveFocus()
passwordBox.selectAll() passwordBox.text = "";
} }
} }
} }
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment