Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(26)

Side by Side Diff: chrome/browser/resources/sync_confirmation/sync_confirmation.css

Issue 1487283005: Implement the new Sync Confirmation dialog on Linux and Windows. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address feedback Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(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 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698