[node-patches] Change in ovirt-node[master]: ui: Show hint on incorrect unlock password

[fabiand at fedoraproject.org]

Fabian Deutsch has uploaded a new change for review.

Change subject: ui: Show hint on incorrect unlock password
......................................................................

ui: Show hint on incorrect unlock password

Previously no feedback was given to th euser in case of an invalid
password on the lock screen. Now a message is displayed and the password
field is reset.

rhbz#911669

Change-Id: I181c8e83907736ae8c59676f973c8ac27fdf5b86
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M scripts/tui/src/ovirt/node/setup/status_page.py
1 file changed, 4 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/42/13942/1

diff --git a/scripts/tui/src/ovirt/node/setup/status_page.py b/scripts/tui/src/ovirt/node/setup/status_page.py
index 987d369..3172a53 100644
--- a/scripts/tui/src/ovirt/node/setup/status_page.py
+++ b/scripts/tui/src/ovirt/node/setup/status_page.py
@@ -125,6 +125,7 @@
             self.logger.info("Locking screen")
             self._lock_dialog = LockDialog()
             self.application.ui.hotkeys_enabled(False)
+            self.widgets.add(self._lock_dialog)
             return self._lock_dialog
         elif "action.unlock" in changes and "password" in changes:
             self.logger.info("UnLocking screen")
@@ -132,6 +133,9 @@
             if pam.authenticate(os.getlogin(), changes["password"]):
                 self._lock_dialog.close()
                 self.application.ui.hotkeys_enabled(True)
+            else:
+                self.application.notice("The provided password was incorrect.")
+                self.widgets["password"].text("")
 
         elif "action.logoff" in changes:
             self.logger.info("Logging off")


--
To view, visit http://gerrit.ovirt.org/13942
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I181c8e83907736ae8c59676f973c8ac27fdf5b86
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>