New issue
Advanced search Search tips

Issue 676565 link

Starred by 2 users
Status: Verified
Owner:
steve...@chromium.org
Closed: Jan 2017
Cc:
tbuck...@chromium.org
bettes@chromium.org
steve...@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug

Blocking:
issue 660597



Sign in to add a comment

MD Settings: Arrangement stuff shows for only one screen

Project Member Reported by michae...@chromium.org, Dec 22 2016 
Chrome Version       : 56.0.2924.28
OS Version: 9000.29.0

The "Arrangement" and "Drag to move a display" explanatory text shows at the top of the chrome://md-settings/display subpage even when there is only one screen (e.g. most of the time on a chromebook).

I dunno if the whole display rectangle thing should show either, since it doesn't do anything or convey anything not already there.

Comment 1 by steve...@chromium.org, Dec 22 2016

Cc: steve...@chromium.org bettes@chromium.org
Owner: tbuck...@chromium.org
Tom, WDYT? I think it's fine as-is since it suggests that this is where multiple monitors will be configured, but we can also hide the text or the section with a single display.

Comment 2 by tbuck...@chromium.org, Jan 6 2017

Labels: -Pri-2 Pri-3
Owner: steve...@chromium.org
I think we can hide the "Drag to move a display" when only one is visible.

Comment 3 by steve...@chromium.org, Jan 9 2017

Labels: M-58

Comment 4 by tbuck...@chromium.org, Jan 9 2017

Labels: Hotlist-MD-Settings-Device

Comment 5 by steve...@chromium.org, Jan 17 2017

Status: Started (was: Assigned)
Project Member

Comment 6 by bugdroid1@chromium.org, Jan 17 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ef9f096a99f5a1b05de1daf1783b32cedf842f26

commit ef9f096a99f5a1b05de1daf1783b32cedf842f26
Author: stevenjb <stevenjb@chromium.org>
Date: Tue Jan 17 22:33:35 2017

MD Settings > Display: Hide arrangement text for single display

BUG= 676565 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2643483002
Cr-Commit-Position: refs/heads/master@{#444166}

[modify] https://crrev.com/ef9f096a99f5a1b05de1daf1783b32cedf842f26/chrome/browser/resources/settings/device_page/display.html
[modify] https://crrev.com/ef9f096a99f5a1b05de1daf1783b32cedf842f26/chrome/browser/resources/settings/device_page/display.js

Comment 7 by steve...@chromium.org, Jan 18 2017

Status: Fixed (was: Started)

Comment 8 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58

Comment 9 by sdantul...@chromium.org, Mar 29 2017

Status: Verified (was: Fixed)
9334.23.0 / 58.0.3029.39
►

Sign in to add a comment