New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 890088 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Oct 2
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression


Participants' hotlists:
Better-Together-Launch-Blockers


Sign in to add a comment

Setup Flow UI right/left margin grow in each successive screen

Project Member Reported by jordynass@chromium.org, Sep 28

Issue description

I believe the margin around the main element (i.e. the 'multidevice-setup' element) goes from 0 in the start page, 40px in the password page, and 112px in the success screen.

khorimoto thinks it is caused by one of the CLs directed at formatting for tablet mode.
 
1 Start Setup.png
394 KB View Download
2 Password.png
349 KB View Download
3 Setup Succeeded.png
354 KB View Download
Cc: jlklein@chromium.org elizabethchiu@chromium.org jordynass@chromium.org shibasheikh@chromium.org jhawkins@chromium.org hansberry@chromium.org jessejames@google.com
Components: UI>Multidevice
Labels: -Type-Bug Type-Bug-Regression
Status: Assigned (was: Untriaged)
Owner: ----
Status: Available (was: Assigned)
Working on another issue right now, so I'm marking as available (this is P1 so if anyone else has bandwidth, they should take it). Glad to take it on later if still open.
Owner: jordynass@chromium.org
Project Member

Comment 4 by bugdroid1@chromium.org, Oct 2

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

commit 5cbcd4751d69ea41b97428ea3fff9710643ecfd7
Author: Jordy Greenblatt <jordynass@chromium.org>
Date: Tue Oct 02 01:18:37 2018

[CrOS MultiDevice] Fix growing margins bug in setup flow

Screenshots from both tablet and standard modes on PixelBook:
https://drive.google.com/drive/folders/18lSB22-hWttyVZxNI7ASAE6m1Qvub8vI?usp=sharing

Bug:  890088 
Change-Id: I9367c87dc85a61b076d0db4ed1997980a308da86
Reviewed-on: https://chromium-review.googlesource.com/1256162
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Commit-Queue: Jordy Greenblatt <jordynass@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595678}
[modify] https://crrev.com/5cbcd4751d69ea41b97428ea3fff9710643ecfd7/ui/webui/resources/cr_components/chromeos/multidevice_setup/multidevice_setup.html

Status: Fixed (was: Available)
Cc: nohle@chromium.org jessejames@chromium.org
Issue 892283 has been merged into this issue.

Sign in to add a comment