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

Side by Side Diff: chrome/browser/resources/chromeos/login/screen_public_account_signin.css

Issue 11308081: cros: Account picker UI for public account. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebased on top of 11419184 Created 8 years 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 | Annotate | Revision Log
OLDNEW
(Empty)
1 /* Copyright (c) 2012 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
6 #public-account-signin {
7 -webkit-box-align: center;
8 display: -webkit-box;
9 }
10
11 #public-account-avartar {
bartfab (slow) 2012/11/28 15:11:38 The avatar size increases here from 160 x 160 in a
xiyuan 2012/12/01 00:24:19 CL refactored to expanding pod in-place and will u
12 display: block;
13 height: 220px;
14 margin: 5px;
15 width: 220px;
16 }
17
18 #public-account-avartar-info-divider {
19 border-left: 1px solid lightgray;
Ivan Korotkov 2012/11/28 21:50:39 Will this work with RTL correctly?
xiyuan 2012/12/01 00:24:19 Yes because the divider div's width is 1px.
20 height: 220px;
21 position: absolute;
22 width: 1px;
23 }
24
25 #public-account-info-pane {
26 height: 220px;
27 margin: 5px;
28 position: relative;
29 width: 250px;
30 }
31
32 #public-account-info-pane .managed-badge {
33 position: absolute;
34 top: 0;
35 }
36 html[dir=ltr] #public-account-info-pane .managed-badge {
37 right: 0;
38 }
39 html[dir=rtl] #public-account-info-pane .managed-badge {
40 left: 0;
41 }
42
43 #public-account-name {
44 color: black;
Nikita (slow) 2012/11/28 18:57:34 Is this really supposed to be black? As far a I re
xiyuan 2012/12/01 00:24:19 I'll let the UX decide. The mock uses black.
45 font-size: 20px;
46 }
47
48 #public-account-text-contents {
49 height: 150px;
50 position: absolute;
51 top: 30px;
52 width: 100%;
53 }
54
55 #public-account-info,
56 #public-account-hint {
57 height: 40%;
58 }
59
60 #public-account-info {
61 margin-top: 20px;
62 }
63
64 #public-account-hint {
65 font-weight: bold;
66 }
67
68 #public-account-enter-button {
69 bottom: 0;
70 display: block;
71 position: absolute;
Ivan Korotkov 2012/11/28 21:50:39 I guess you can simplify a bit if you add 'right:
xiyuan 2012/12/01 00:24:19 Done in its new incarnation, i.e. as ".side-pane-c
72 }
73 html[dir=ltr] #public-account-enter-button {
Nikita (slow) 2012/11/28 18:57:34 nit: add extra lines between styles
xiyuan 2012/12/01 00:24:19 Removed now.
74 right: 0;
75 }
76 html[dir=rtl] #public-account-enter-button {
Nikita (slow) 2012/11/28 18:57:34 nit: same here
xiyuan 2012/12/01 00:24:19 Done in its new incarnation, i.e. as rtl rule for
77 left: 0;
78 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698