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

Issue 704059 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

ChromeOS OOBE: CFM Welcome screen looks strange

Project Member Reported by alemate@chromium.org, Mar 22 2017

Issue description

The first attachment is how it looks now (because the box with header text is center-aligned).

The second screenshot is what we will get if we also align the text itself to the center.

Also this text depends on device locale, so it may grow/shrink.


What do you think? How should we adjust text display for the welcome screen?

 
wBiL7gvenoc.png
71.6 KB View Download
d1soqXb0RAV.png
71.7 KB View Download
Is it a must to use this string? We can't just use "Welcome"?
If we really need to stick with this text (it's too long as a welcome header in my opinion), let's make the type Roboto light 32sp, 40sp for leading. Please match with my mock. 
01 - Chromebox.png
416 KB View Download

Comment 3 Deleted

Cc: elizabethchiu@chromium.org
Components: UI>Shell>OOBE
Labels: M-58
I think it's probably fine to keep the text, but let's update to the font size Elizabeth specified above.
What do you mean by "leading" ? I interpreted it as "the first line font size must be 40px, and all the other lines font size must be 32px". Here are screenshots for "font-size: 32px; font-align: center;" and 40px for the first line (see below).

The first four screenshots are examples of "Welcome" screen for CfM devices in different languages.



Comment 7 Deleted

Screenshot 2017-04-05 at 11.05.41 PM.png
520 KB View Download
Screenshot 2017-04-05 at 11.06.19 PM.png
520 KB View Download
Screenshot 2017-04-05 at 11.06.36 PM.png
518 KB View Download
Screenshot 2017-04-05 at 11.07.07 PM.png
514 KB View Download
Please note that due to the different lengths of words and string in total, the picture looks very different.


If I reduce heading width to 500px to make English looks like your example, the result will be different (see below).
Screenshot 2017-04-05 at 11.07.31 PM.png
520 KB View Download
Screenshot 2017-04-05 at 11.07.44 PM.png
522 KB View Download
Screenshot 2017-04-05 at 11.08.03 PM.png
518 KB View Download
Screenshot 2017-04-05 at 11.08.26 PM.png
511 KB View Download
Owner: elizabethchiu@chromium.org
Please note that text in #14 occupies three lines, which moves the illustration down. So we probably  do not want to reduce message width.


WDYT?
Please note that I did not update font-weight, because we were already using font-weight: 300 (which is light). The last 8 screenshots were taken on a device, so font should be correct.
Owner: alemate@chromium.org
Hey Alexander, I believe leading in this case refers to the line-height, specifically the height between the base lines. So all the text should be size 32sp, with a line-height of 40sp.
(Elizabeth, correct me if that's wrong).

Comment 20 Deleted

Here are the screenshots.
Screenshot 2017-04-07 at 10.22.29 PM.png
514 KB View Download
Screenshot 2017-04-07 at 10.22.39 PM.png
517 KB View Download
Screenshot 2017-04-07 at 10.22.50 PM.png
515 KB View Download
Screenshot 2017-04-07 at 10.23.52 PM.png
511 KB View Download
Status: Started (was: Assigned)
The English one looks weird. Is it possible to match the screenshot I sent? Take out the period at the end, and avoid having "up" hanging on the second line. The title should not go beyond the button on the left. Would you mind to make the left and right margin of the title to be 180?
The title never gets beyond the buttons.
See the screenshots of container borders attached.
CfM-welcome-offsets-screenshot.jpg
50.4 KB View Download
CfM-welcome-offsets-screenshot1.jpg
57.9 KB View Download
If I set margins to 80 (text margin) + 22 (dialog margin) + 3 (border) = 125, the first screen of CfM will look like this:

Screenshot from 2017-04-10 23:35:57.png
50.3 KB View Download
But French ( and other 13 languages) will look like this:

Screenshot from 2017-04-10 23:36:49.png
52.8 KB View Download
While Arabian will look like this:
Screenshot from 2017-04-10 23:40:11.png
44.0 KB View Download
Project Member

Comment 28 by bugdroid1@chromium.org, Apr 11 2017

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

commit 48d63103e76cc388b4fe5151b2cb66d8af56bbaf
Author: alemate <alemate@chromium.org>
Date: Tue Apr 11 08:16:34 2017

ChromeOS OOBE: Update Welcome dialog to better accomodate CfM messages.

CfM uses longer elcoming text so this Cl makes the presentation better.

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

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

[modify] https://crrev.com/48d63103e76cc388b4fe5151b2cb66d8af56bbaf/chrome/browser/resources/chromeos/login/oobe_welcome_dialog.css

Labels: Merge-Request-58 OS-Chrome
Status: Fixed (was: Started)
As discussed with zalcorn@ offline, we cannot update strings for M58, so we will just narrow the width of the header by some reasonable amount.

So this is fixed.
Project Member

Comment 30 by sheriffbot@chromium.org, Apr 11 2017

Labels: -Merge-Request-58 Merge-Review-58 Hotlist-Merge-Review
This bug requires manual review: We are only 13 days from stable.
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
For the French one, looks like 1-2 words can go to the first line. The title should be able to fit just 2 lines. As for the Arabian screen, can we add the same spacing as English version between the text and the icon in the buttons at the bottom?
> For the French one, looks like 1-2 words can go to the first line. 

We cannot do any hacks for one language.
We can only implement some general rule.
Right now the rule (from /cros-oobe-spec) says: the baseline of the first line of the header text starts 152 pixels from the top of the dialog.

The width of the header was unspecified, but on the last screenshots it is "768-(125*2) = 568px".

The top of the illustration is bound to the baseline of the first line of the header.

We can implement another layout, but we need a design for it.
Thanks for letting me know. It's because I did not know that we need other headers for CFM in the beginning. For chromebooks, we only use "welcome" and that works well in different languages.

If this is the case, I need all the strings for different languages and make sure all of them look good. Would you mind to provide me all the headers in different languages? 
Labels: HotRod
Labels: -Hotlist-Merge-Review -M-58 -Merge-Review-58 M-59
No need to update M58.

Comment 37 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment