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

Side by Side Diff: chrome/browser/resources/settings/people_page/people_page.js

Issue 2793463002: MD Settings: Restore focus after closing various dialogs. (Closed)
Patch Set: Nits. Created 3 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * @fileoverview 6 * @fileoverview
7 * 'settings-people-page' is the settings page containing sign-in settings. 7 * 'settings-people-page' is the settings page containing sign-in settings.
8 */ 8 */
9 Polymer({ 9 Polymer({
10 is: 'settings-people-page', 10 is: 'settings-people-page',
(...skipping 290 matching lines...) Expand 10 before | Expand all | Expand 10 after
301 }, 301 },
302 302
303 /** @private */ 303 /** @private */
304 onSigninTap_: function() { 304 onSigninTap_: function() {
305 this.syncBrowserProxy_.startSignIn(); 305 this.syncBrowserProxy_.startSignIn();
306 }, 306 },
307 307
308 /** @private */ 308 /** @private */
309 onDisconnectClosed_: function() { 309 onDisconnectClosed_: function() {
310 this.showDisconnectDialog_ = false; 310 this.showDisconnectDialog_ = false;
311 this.$$('#disconnectButton').focus();
312
311 if (settings.getCurrentRoute() == settings.Route.SIGN_OUT) 313 if (settings.getCurrentRoute() == settings.Route.SIGN_OUT)
312 settings.navigateToPreviousRoute(); 314 settings.navigateToPreviousRoute();
313 this.fire('signout-dialog-closed'); 315 this.fire('signout-dialog-closed');
314 }, 316 },
315 317
316 /** @private */ 318 /** @private */
317 onDisconnectTap_: function() { 319 onDisconnectTap_: function() {
318 settings.navigateTo(settings.Route.SIGN_OUT); 320 settings.navigateTo(settings.Route.SIGN_OUT);
319 }, 321 },
320 322
(...skipping 109 matching lines...) Expand 10 before | Expand all | Expand 10 after
430 }, 432 },
431 433
432 /** @private */ 434 /** @private */
433 onImportDataTap_: function() { 435 onImportDataTap_: function() {
434 settings.navigateTo(settings.Route.IMPORT_DATA); 436 settings.navigateTo(settings.Route.IMPORT_DATA);
435 }, 437 },
436 438
437 /** @private */ 439 /** @private */
438 onImportDataDialogClosed_: function() { 440 onImportDataDialogClosed_: function() {
439 settings.navigateToPreviousRoute(); 441 settings.navigateToPreviousRoute();
442 this.$.importDataDialogTrigger.focus();
440 }, 443 },
441 // </if> 444 // </if>
442 445
443 /** 446 /**
444 * @private 447 * @private
445 * @param {string} domain 448 * @param {string} domain
446 * @return {string} 449 * @return {string}
447 */ 450 */
448 getDisconnectExplanationHtml_: function(domain) { 451 getDisconnectExplanationHtml_: function(domain) {
449 // <if expr="not chromeos"> 452 // <if expr="not chromeos">
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
519 522
520 /** 523 /**
521 * @param {!settings.SyncStatus} syncStatus 524 * @param {!settings.SyncStatus} syncStatus
522 * @return {boolean} Whether to show the "Sign in to Chrome" button. 525 * @return {boolean} Whether to show the "Sign in to Chrome" button.
523 * @private 526 * @private
524 */ 527 */
525 showSignin_: function(syncStatus) { 528 showSignin_: function(syncStatus) {
526 return !!syncStatus.signinAllowed && !syncStatus.signedIn; 529 return !!syncStatus.signinAllowed && !syncStatus.signedIn;
527 }, 530 },
528 }); 531 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698