ChromeOS OOBE: update MD-OOBE after UI review.

ChromeOS OOBE: update MD-OOBE after UI review.

This CL updates styles for ChromeOS OOBE screens after UI review.

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

Review-Url: https://codereview.chromium.org/2635493004
Cr-Commit-Position: refs/heads/master@{#443966}
Committed: https://chromium.googlesource.com/chromium/src/+/571fb9dccbf8f0d39f708a7639eef95e9bd7f1ac

[Alexander Alekseev, 2017-01-13 23:19:46]

Description was changed from

==========
ChromeOS OOBE: update MD-OOBE after UI review.

This CL updates styles for ChromeOS OOBE screens after UI review.

BUG=604119
==========

to

==========
ChromeOS OOBE: update MD-OOBE after UI review.

This CL updates styles for ChromeOS OOBE screens after UI review.

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

[Alexander Alekseev, 2017-01-13 23:23:12]

alemate@chromium.org changed reviewers:
+ achuith@chromium.org

[Alexander Alekseev, 2017-01-13 23:23:13]

Please review.

[Alexander Alekseev, 2017-01-13 23:23:18]

The CQ bit was checked by alemate@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-13 23:24:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[achuithb, 2017-01-13 23:38:30]

On 2017/01/13 23:23:13, Alexander Alekseev wrote:
> Please review.

I can lg this, but tbh, my CSS is super weak. I think someone like michaelpg
might be a better bet.

[Alexander Alekseev, 2017-01-13 23:49:31]

alemate@chromium.org changed reviewers:
+ michaelpg@chromium.org
- achuith@chromium.org

[Alexander Alekseev, 2017-01-13 23:49:32]

-achuith
+michaelpg

Please review.

[commit-bot: I haz the power, 2017-01-14 00:35:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-14 00:35:38]

Dry run: This issue passed the CQ dry run.

[Alexander Alekseev, 2017-01-14 03:17:02]

The CQ bit was checked by alemate@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-14 03:17:13]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-14 04:18:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-14 04:18:21]

Dry run: This issue passed the CQ dry run.

[michaelpg, 2017-01-16 17:57:47]

lgtm

https://codereview.chromium.org/2635493004/diff/20001/chrome/browser/resource...
File chrome/browser/resources/chromeos/login/oobe_eula.css (right):

https://codereview.chromium.org/2635493004/diff/20001/chrome/browser/resource...
chrome/browser/resources/chromeos/login/oobe_eula.css:27:
--paper-checkbox-checked-color: rgb(66, 133, 244); /* #4285f4 */
nit: the hex comment isn't helpful. if you really need to know you can just
google it: https://www.google.com/search?q=rgb(66%2C%20133%2C%20244)%20to%20hex

https://codereview.chromium.org/2635493004/diff/20001/ui/login/screen_contain...
File ui/login/screen_container.css (right):

https://codereview.chromium.org/2635493004/diff/20001/ui/login/screen_contain...
ui/login/screen_container.css:75: background-color: rgba(20, 29, 40, .8); /*
#141D28 80% */
same

[Alexander Alekseev, 2017-01-17 00:39:16]

https://codereview.chromium.org/2635493004/diff/20001/chrome/browser/resource...
File chrome/browser/resources/chromeos/login/oobe_eula.css (right):

https://codereview.chromium.org/2635493004/diff/20001/chrome/browser/resource...
chrome/browser/resources/chromeos/login/oobe_eula.css:27:
--paper-checkbox-checked-color: rgb(66, 133, 244); /* #4285f4 */
On 2017/01/16 17:57:47, michaelpg wrote:
> nit: the hex comment isn't helpful. if you really need to know you can just
> google it:
https://www.google.com/search?q=rgb(66%2C%20133%2C%20244)%20to%20hex

This is mainly for grepping it later as our designers specify colors in hex.

[Alexander Alekseev, 2017-01-17 00:39:27]

The CQ bit was checked by alemate@chromium.org

[commit-bot: I haz the power, 2017-01-17 00:39:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-17 01:33:09]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484613567166910,
"parent_rev": "35745b4cb5b0ef6a58ddc249612e93893e513e65", "commit_rev":
"571fb9dccbf8f0d39f708a7639eef95e9bd7f1ac"}

[commit-bot: I haz the power, 2017-01-17 01:33:58]

Description was changed from

==========
ChromeOS OOBE: update MD-OOBE after UI review.

This CL updates styles for ChromeOS OOBE screens after UI review.

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

to

==========
ChromeOS OOBE: update MD-OOBE after UI review.

This CL updates styles for ChromeOS OOBE screens after UI review.

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

Review-Url: https://codereview.chromium.org/2635493004
Cr-Commit-Position: refs/heads/master@{#443966}
Committed:
https://chromium.googlesource.com/chromium/src/+/571fb9dccbf8f0d39f708a7639ee...
==========

[commit-bot: I haz the power, 2017-01-17 01:33:59]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/571fb9dccbf8f0d39f708a7639ee...