Chromium Code Reviews Chromium Code Reviews
Issue 1487283005:
Implement the new Sync Confirmation dialog on Linux and Windows. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| OLD | NEW |
|---|---|
| (Empty) | |
| 1 /* Copyright 2015 The Chromium Authors. All rights reserved. | |
| 2 * Use of this source code is governed by a BSD-style license that can be | |
| 3 * found in the LICENSE file. */ | |
| 4 | |
| 5 body { | |
| 6 font-size: 100%; | |
|
Dan Beam
2016/01/22 01:26:58
wait, why are you doing this? to override this ru
anthonyvd
2016/01/22 17:19:56
Yes, although I should have adjusted my em values
Dan Beam
2016/03/04 00:26:48
i think the <body> (default font-size) should be 8
| |
| 7 margin: 0; | |
| 8 padding: 0; | |
| 9 } | |
| 10 | |
| 11 a { | |
| 12 color: rgb(85, 149, 254); | |
| 13 text-decoration: none; | |
| 14 } | |
| 15 | |
| 16 .picture img { | |
| 17 border-radius: 50%; | |
| 18 max-height: 100%; | |
| 19 max-width: 100%; | |
| 20 } | |
| 21 | |
| 22 .container { | |
| 23 background-color: white; | |
| 24 height: 351px; | |
| 25 overflow: hidden; | |
| 26 width: 448px; | |
|
Dan Beam
2016/01/22 01:26:58
where are these dimensions from? do they assume E
anthonyvd
2016/01/22 17:19:56
They're from our designer mocks. I don't think the
| |
| 27 } | |
| 28 | |
| 29 .top-title-bar { | |
| 30 -webkit-padding-start: 24px; | |
|
Dan Beam
2016/01/22 01:26:58
this only adds padding on one end (the start)
anthonyvd
2016/01/22 17:19:56
Done.
| |
| 31 align-items: center; | |
| 32 border-bottom: 1px solid lightgray; | |
| 33 color: #333; | |
| 34 display: flex; | |
| 35 font-size: 1em; | |
| 36 height: 56px; | |
| 37 } | |
| 38 | |
| 39 .details { | |
| 40 height: 250px; | |
| 41 padding: 20px 24px; | |
| 42 } | |
| 43 | |
| 44 .sync-message { | |
| 45 color: #595959; | |
| 46 font-size: 0.8125em; | |
| 47 line-height: 150%; | |
| 48 padding-bottom: 32px; | |
| 49 } | |
| 50 | |
| 51 .picture-container { | |
| 52 align-items: center; | |
| 53 display: flex; | |
| 54 justify-content: center; | |
| 55 padding-bottom: 32px; | |
| 56 } | |
| 57 | |
| 58 .picture { | |
| 59 height: 96px; | |
| 60 position: relative; | |
| 61 width: 96px; | |
| 62 } | |
| 63 | |
| 64 #profile-picture, | |
| 65 .checkmark-bubble { | |
| 66 position: absolute; | |
| 67 } | |
| 68 | |
| 69 .checkmark-bubble { | |
|
Dan Beam
2016/01/22 01:26:58
are you duplicating <paper-checkbox>?
anthonyvd
2016/01/22 17:19:56
I'm not. This isn't a checkbox, it's the white che
| |
| 70 background-color: white; | |
| 71 background-image: url(//resources/images/check_circle.svg); | |
|
Dan Beam
2016/01/22 01:26:57
nit: use a local path (i.e ../../../../ui/webui/re
anthonyvd
2016/01/22 17:19:56
background-image: url(../../../../ui/webui/resourc
| |
| 72 background-size: 100%; | |
| 73 border: 1px solid white; | |
| 74 border-radius: 50%; | |
| 75 display: inline-block; | |
| 76 height: 30px; | |
| 77 left: 64px; | |
|
Dan Beam
2016/01/22 01:26:58
still want to see how this is used
fwiw: blink do
anthonyvd
2016/01/22 17:19:56
See above comment for a screenshot and some explan
| |
| 78 top: 66px; | |
| 79 width: 30px; | |
| 80 } | |
| 81 | |
| 82 .action-container { | |
| 83 align-items: baseline; | |
| 84 display: flex; | |
| 85 justify-content: flex-end; | |
| 86 } | |
| 87 | |
| 88 paper-button { | |
| 89 font-size: 0.8125em; | |
| 90 padding-left: 12px; | |
| 91 padding-right: 12px; | |
| 92 } | |
| 93 | |
| 94 #confirmButton { | |
| 95 background-color: rgb(66, 133, 244); | |
| 96 color: white; | |
| 97 } | |
| 98 | |
| 99 #undoButton { | |
| 100 color: #5A5A5A; | |
| 101 } | |
| OLD | NEW |
