[gnome-shell/uajain/adapt-user-avatar-part2: 55/62] authPrompt: Connect signals to both text and password entries



commit 017444063e7f9d87323a51424a5e4a5df137c44e
Author: Georges Basile Stavracas Neto <georges stavracas gmail com>
Date:   Thu Feb 6 17:27:20 2020 -0300

    authPrompt: Connect signals to both text and password entries
    
    This is a regression from the transition to password entry. Both
    entries need to be connected to the relevant signals, otherwise
    username-based login won't ever work.
    
    Connect both the text and the password entries.
    
    https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/922

 js/gdm/authPrompt.js | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)
---
diff --git a/js/gdm/authPrompt.js b/js/gdm/authPrompt.js
index 1a36b994e2..d29414004b 100644
--- a/js/gdm/authPrompt.js
+++ b/js/gdm/authPrompt.js
@@ -159,14 +159,16 @@ var AuthPrompt = GObject.registerClass({
         this._mainBox.add_child(this._entry);
         this._entry.grab_key_focus();
 
-        this._entry.clutter_text.connect('text-changed', () => {
-            if (!this._userVerifier.hasPendingMessages)
-                this._fadeOutMessage();
-        });
-
-        this._entry.clutter_text.connect('activate', () => {
-            if (this._entry.reactive)
-                this.emit('next');
+        [this._textEntry, this._passwordEntry].forEach(entry => {
+            entry.clutter_text.connect('text-changed', () => {
+                if (!this._userVerifier.hasPendingMessages)
+                    this._fadeOutMessage();
+            });
+
+            entry.clutter_text.connect('activate', () => {
+                if (entry.reactive)
+                    this.emit('next');
+            });
         });
 
         this._defaultButtonWell = new St.Widget({


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]