From 6daeb048afff243377ddc889f46395de880cf9d3 Mon Sep 17 00:00:00 2001 From: Georges Basile Stavracas Neto Date: Thu, 6 Feb 2020 17:27:20 -0300 Subject: [PATCH] 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 | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/js/gdm/authPrompt.js b/js/gdm/authPrompt.js index 1a36b994e..d29414004 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._textEntry, this._passwordEntry].forEach(entry => { + 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'); + entry.clutter_text.connect('activate', () => { + if (entry.reactive) + this.emit('next'); + }); }); this._defaultButtonWell = new St.Widget({