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

Side by Side Diff: chrome/browser/resources/settings/device_page/display.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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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-display' is the settings subpage for display settings. 7 * 'settings-display' is the settings subpage for display settings.
8 */ 8 */
9 9
10 cr.define('settings.display', function() { 10 cr.define('settings.display', function() {
(...skipping 468 matching lines...) Expand 10 before | Expand all | Expand 10 after
479 * @private 479 * @private
480 */ 480 */
481 onOverscanTap_: function(e) { 481 onOverscanTap_: function(e) {
482 e.preventDefault(); 482 e.preventDefault();
483 this.overscanDisplayId = this.selectedDisplay.id; 483 this.overscanDisplayId = this.selectedDisplay.id;
484 this.showOverscanDialog_(true); 484 this.showOverscanDialog_(true);
485 }, 485 },
486 486
487 /** @private */ 487 /** @private */
488 onCloseOverscanDialog_: function() { 488 onCloseOverscanDialog_: function() {
489 this.$$('#overscan button').focus(); 489 cr.ui.focusWithoutInk(assert(this.$$('#overscan button')));
490 }, 490 },
491 491
492 /** @private */ 492 /** @private */
493 updateDisplayInfo_: function() { 493 updateDisplayInfo_: function() {
494 var displayIds = ''; 494 var displayIds = '';
495 var primaryDisplay = undefined; 495 var primaryDisplay = undefined;
496 var selectedDisplay = undefined; 496 var selectedDisplay = undefined;
497 for (var i = 0; i < this.displays.length; ++i) { 497 for (var i = 0; i < this.displays.length; ++i) {
498 var display = this.displays[i]; 498 var display = this.displays[i];
499 if (displayIds) 499 if (displayIds)
(...skipping 16 matching lines...) Expand all
516 }, 516 },
517 517
518 /** @private */ 518 /** @private */
519 setPropertiesCallback_: function() { 519 setPropertiesCallback_: function() {
520 if (chrome.runtime.lastError) { 520 if (chrome.runtime.lastError) {
521 console.error( 521 console.error(
522 'setDisplayProperties Error: ' + chrome.runtime.lastError.message); 522 'setDisplayProperties Error: ' + chrome.runtime.lastError.message);
523 } 523 }
524 }, 524 },
525 }); 525 });
OLDNEW
« no previous file with comments | « chrome/browser/resources/settings/device_page/display.html ('k') | chrome/browser/resources/settings/device_page/storage.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698