[gnome-shell/uajain/adapt-user-avatar-part2: 20/22] authPrompt: Only spin on password entries
- From: Georges Basile Stavracas Neto <gbsneto src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell/uajain/adapt-user-avatar-part2: 20/22] authPrompt: Only spin on password entries
- Date: Wed, 12 Feb 2020 18:00:39 +0000 (UTC)
commit 3f444ebce77d9dec03c091b22482307508cd0674
Author: Georges Basile Stavracas Neto <georges stavracas gmail com>
Date: Tue Feb 11 10:33:02 2020 -0300
authPrompt: Only spin on password entries
Spinning while typing the password is a bit off-putting, and
inconsistent with how regular authentication behaves.
Only spin the spinner after typing the password.
https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/922
js/gdm/authPrompt.js | 10 +++++++---
js/gdm/loginDialog.js | 1 -
2 files changed, 7 insertions(+), 4 deletions(-)
---
diff --git a/js/gdm/authPrompt.js b/js/gdm/authPrompt.js
index c053d55230..b7bf38d485 100644
--- a/js/gdm/authPrompt.js
+++ b/js/gdm/authPrompt.js
@@ -157,8 +157,9 @@ var AuthPrompt = GObject.registerClass({
});
entry.clutter_text.connect('activate', () => {
+ let shouldSpin = entry === this._passwordEntry;
if (entry.reactive)
- this._activateNext();
+ this._activateNext(shouldSpin);
});
});
@@ -177,9 +178,12 @@ var AuthPrompt = GObject.registerClass({
this._defaultButtonWell.add_child(this._spinner);
}
- _activateNext() {
+ _activateNext(shouldSpin) {
this.updateSensitivity(false);
- this.startSpinning();
+
+ if (shouldSpin)
+ this.startSpinning();
+
if (this._queryingService)
this._userVerifier.answerQuery(this._queryingService, this._entry.text);
else
diff --git a/js/gdm/loginDialog.js b/js/gdm/loginDialog.js
index 3fbf0659e7..a45b3489b8 100644
--- a/js/gdm/loginDialog.js
+++ b/js/gdm/loginDialog.js
@@ -928,7 +928,6 @@ var LoginDialog = GObject.registerClass({
let answer = this._authPrompt.getAnswer();
this._user = this._userManager.get_user(answer);
this._authPrompt.clear();
- this._authPrompt.startSpinning();
this._authPrompt.begin({ userName: answer });
this._updateCancelButton();
});
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]