Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(50)

Unified Diff: chrome/browser/resources/settings/people_page/password_prompt_dialog.js

Issue 2441043004: Accessibility fixes for quick unlock settings. (Closed)
Patch Set: Address comments, fix test Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/settings/people_page/password_prompt_dialog.js
diff --git a/chrome/browser/resources/settings/people_page/password_prompt_dialog.js b/chrome/browser/resources/settings/people_page/password_prompt_dialog.js
index 7dae95142d97ae14d155c4d897468b608f94a32a..e7c23575511f4dda5ebc0877ebc8f8ef8931cb6d 100644
--- a/chrome/browser/resources/settings/people_page/password_prompt_dialog.js
+++ b/chrome/browser/resources/settings/people_page/password_prompt_dialog.js
@@ -81,6 +81,17 @@ Polymer({
},
},
+ /** @override */
+ ready: function() {
+ // The first time the dialog is shown, the password prompt does not
+ // auto-focus itself. However, if the dialog has been created already
+ // and is only being redisplayed, the password input will correctly
+ // capture focus by itself.
+ setTimeout(function() {
+ this.$.passwordInput.focus();
Dan Beam 2016/10/26 22:52:09 don't start a focus storm
jdufault 2016/10/26 23:21:08 Strange, using autofocus works as expected now. I'
+ }.bind(this));
+ },
+
/**
* Open up the dialog. This will wait until the dialog has loaded before
* opening it.

Powered by Google App Engine
This is Rietveld 408576698