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

Unified Diff: chrome/browser/resources/options2/chromeos/display_options.js

Issue 10828005: Show "displays" section in the options page even for 1 display. It happens in case of mirroring. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 5 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/options2/chromeos/display_options.js
diff --git a/chrome/browser/resources/options2/chromeos/display_options.js b/chrome/browser/resources/options2/chromeos/display_options.js
index e1705a4de8f0a2e24d1dd12d0097d2c1bd76e605..5c4b5c7406216dd46c9b3e190fb69f6597418454 100644
--- a/chrome/browser/resources/options2/chromeos/display_options.js
+++ b/chrome/browser/resources/options2/chromeos/display_options.js
@@ -230,19 +230,22 @@ cr.define('options', function() {
var width = this.displays_[0].width * VISUAL_SCALE;
var height = this.displays_[0].height * VISUAL_SCALE;
+ // Sometimes the system provies only one display in case of mirroring.
+ // We show at least 2 display rectangles to have the "mirroring" looking.
oshima 2012/07/25 07:53:41 We only support up to 2 displays now, and there is
oshima 2012/07/25 07:54:25 should read: "there is only one display from chro
Jun Mukai 2012/07/25 08:14:43 Done.
+ var num_displays = Math.max(2, this.displays_.length);
+
this.displays_view_.style.height =
- height + this.displays_.length * 2 + 'px';
+ height + num_displays * 2 + 'px';
- for (var i = 0; i < this.displays_.length; i++) {
+ for (var i = 0; i < num_displays; i++) {
var div = document.createElement('div');
- this.displays_[i].div = div;
div.className = 'displays-display';
div.style.top = i * 2 + 'px';
div.style.left = i * 2 + 'px';
div.style.width = width + 'px';
div.style.height = height + 'px';
div.style.zIndex = i;
- if (i == this.displays_.length - 1)
+ if (i == num_displays - 1)
div.className += ' displays-primary';
this.displays_view_.appendChild(div);
}
@@ -330,9 +333,6 @@ cr.define('options', function() {
this.displays_ = displays;
- if (this.displays_.length <= 1)
- return;
-
this.resetDisplaysView_();
if (this.mirroring_)
this.layoutMirroringDisplays_();

Powered by Google App Engine
This is Rietveld 408576698