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

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

Issue 2842303004: MD Settings: update dialogs to focus without ink when using mouse (Closed)
Patch Set: merge Created 3 years, 7 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 238 matching lines...) Expand 10 before | Expand all | Expand 10 after
249 // </if> 249 // </if>
250 250
251 /** @private */ 251 /** @private */
252 onSigninTap_: function() { 252 onSigninTap_: function() {
253 this.syncBrowserProxy_.startSignIn(); 253 this.syncBrowserProxy_.startSignIn();
254 }, 254 },
255 255
256 /** @private */ 256 /** @private */
257 onDisconnectClosed_: function() { 257 onDisconnectClosed_: function() {
258 this.showDisconnectDialog_ = false; 258 this.showDisconnectDialog_ = false;
259 this.$$('#disconnectButton').focus(); 259 cr.ui.focusWithoutInk(assert(this.$$('#disconnectButton')));
260 260
261 if (settings.getCurrentRoute() == settings.Route.SIGN_OUT) 261 if (settings.getCurrentRoute() == settings.Route.SIGN_OUT)
262 settings.navigateToPreviousRoute(); 262 settings.navigateToPreviousRoute();
263 this.fire('signout-dialog-closed'); 263 this.fire('signout-dialog-closed');
264 }, 264 },
265 265
266 /** @private */ 266 /** @private */
267 onDisconnectTap_: function() { 267 onDisconnectTap_: function() {
268 settings.navigateTo(settings.Route.SIGN_OUT); 268 settings.navigateTo(settings.Route.SIGN_OUT);
269 }, 269 },
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
356 }, 356 },
357 357
358 /** @private */ 358 /** @private */
359 onImportDataTap_: function() { 359 onImportDataTap_: function() {
360 settings.navigateTo(settings.Route.IMPORT_DATA); 360 settings.navigateTo(settings.Route.IMPORT_DATA);
361 }, 361 },
362 362
363 /** @private */ 363 /** @private */
364 onImportDataDialogClosed_: function() { 364 onImportDataDialogClosed_: function() {
365 settings.navigateToPreviousRoute(); 365 settings.navigateToPreviousRoute();
366 this.$.importDataDialogTrigger.focus(); 366 cr.ui.focusWithoutInk(assert(this.$.importDataDialogTrigger));
367 }, 367 },
368 // </if> 368 // </if>
369 369
370 /** 370 /**
371 * @private 371 * @private
372 * @param {string} domain 372 * @param {string} domain
373 * @return {string} 373 * @return {string}
374 */ 374 */
375 getDisconnectExplanationHtml_: function(domain) { 375 getDisconnectExplanationHtml_: function(domain) {
376 // <if expr="not chromeos"> 376 // <if expr="not chromeos">
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
446 446
447 /** 447 /**
448 * @param {!settings.SyncStatus} syncStatus 448 * @param {!settings.SyncStatus} syncStatus
449 * @return {boolean} Whether to show the "Sign in to Chrome" button. 449 * @return {boolean} Whether to show the "Sign in to Chrome" button.
450 * @private 450 * @private
451 */ 451 */
452 showSignin_: function(syncStatus) { 452 showSignin_: function(syncStatus) {
453 return !!syncStatus.signinAllowed && !syncStatus.signedIn; 453 return !!syncStatus.signinAllowed && !syncStatus.signedIn;
454 }, 454 },
455 }); 455 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698