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

Side by Side Diff: chrome/browser/resources/chromeos/wallpaper_manager/css/wallpaper_manager.css

Issue 10919253: Fix img misalign and hide "by" when nothing selected in new wallpaepr picker ui (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: This patch for issue 148751(hide "by" when nothing selected)" Created 8 years, 3 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | chrome/browser/resources/chromeos/wallpaper_manager/js/wallpaper_manager.js » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* Copyright (c) 2012 The Chromium Authors. All rights reserved. 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 2 * Use of this source code is governed by a BSD-style license that can be
3 * found in the LICENSE file. */ 3 * found in the LICENSE file. */
4 4
5 html { 5 html {
6 height: 100%; 6 height: 100%;
7 } 7 }
8 8
9 body { 9 body {
10 background-color: #f1f1f1; 10 background-color: #f1f1f1;
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
114 114
115 .image-picker [role=listitem] { 115 .image-picker [role=listitem] {
116 border: 1px solid rgba(0, 0, 0, 0.15); 116 border: 1px solid rgba(0, 0, 0, 0.15);
117 border-radius: 4px; 117 border-radius: 4px;
118 height: 80px; 118 height: 80px;
119 margin: 10px; 119 margin: 10px;
120 padding: 2px; 120 padding: 2px;
121 width: 128px; 121 width: 128px;
122 } 122 }
123 123
124 /* The width and height of img tag need to be set here. Otherwise, incorrect
125 * image size may be used when layout. Some images may be misaligned as a
126 * result. See http://code.google.com/p/chromium/issues/detail?id=148480 for
127 * screen shot.
flackr 2012/09/13 02:34:27 This could potentially show images at the wrong as
bshe 2012/09/13 03:45:48 Yes. All the thumbnails are downloaded from server
128 */
129 .image-picker img {
130 height: 80px;
131 width: 128px;
132 }
133
124 .image-picker [selected][lead] { 134 .image-picker [selected][lead] {
125 border: 2px solid rgb(0, 102, 204); 135 border: 2px solid rgb(0, 102, 204);
126 padding: 1px; 136 padding: 1px;
127 } 137 }
128 138
129 .wallpaper-attr { 139 #wallpaper-attr {
flackr 2012/09/13 02:34:27 I don't think there's any need to abbreviate this,
bshe 2012/09/13 03:45:48 Done.
130 -webkit-box-orient: vertical; 140 -webkit-box-orient: vertical;
131 display: -webkit-box; 141 display: -webkit-box;
132 } 142 }
133 143
144 #wallpaper-attr[hidden] {
flackr 2012/09/13 02:34:27 perhaps just *[hidden]?
bshe 2012/09/13 03:45:48 Done.
145 display: none;
146 }
147
134 .bottom-bar { 148 .bottom-bar {
135 -webkit-box-align: center; 149 -webkit-box-align: center;
136 -webkit-box-orient: horizontal; 150 -webkit-box-orient: horizontal;
137 -webkit-box-pack: end; 151 -webkit-box-pack: end;
138 display: -webkit-box; 152 display: -webkit-box;
139 height: 30px; 153 height: 30px;
140 padding: 10px 15px; 154 padding: 10px 15px;
141 } 155 }
142 156
143 .bottom-bar > .spacer { 157 .bottom-bar > .spacer {
144 -webkit-box-flex: 1; 158 -webkit-box-flex: 1;
145 } 159 }
146 160
147 /* Custom wallpaper thumbnail container. */ 161 /* Custom wallpaper thumbnail container. */
148 #preview { 162 #preview {
149 -webkit-box-shadow: 0 4px 16px rgba(0, 0, 0, 0.2); 163 -webkit-box-shadow: 0 4px 16px rgba(0, 0, 0, 0.2);
150 height: 80px; 164 height: 80px;
151 width: 128px; 165 width: 128px;
152 } 166 }
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/resources/chromeos/wallpaper_manager/js/wallpaper_manager.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698