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

Unified 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/resources/chromeos/wallpaper_manager/js/wallpaper_manager.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/chromeos/wallpaper_manager/css/wallpaper_manager.css
diff --git a/chrome/browser/resources/chromeos/wallpaper_manager/css/wallpaper_manager.css b/chrome/browser/resources/chromeos/wallpaper_manager/css/wallpaper_manager.css
index 158cf8ef6377b1480883dc3456a817d4f364669c..67fa60a14ed83321fbac3e9415d12e5a430d2ab6 100644
--- a/chrome/browser/resources/chromeos/wallpaper_manager/css/wallpaper_manager.css
+++ b/chrome/browser/resources/chromeos/wallpaper_manager/css/wallpaper_manager.css
@@ -121,16 +121,30 @@ div.sidebar-splitter {
width: 128px;
}
+/* The width and height of img tag need to be set here. Otherwise, incorrect
+ * image size may be used when layout. Some images may be misaligned as a
+ * result. See http://code.google.com/p/chromium/issues/detail?id=148480 for
+ * 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
+ */
+.image-picker img {
+ height: 80px;
+ width: 128px;
+}
+
.image-picker [selected][lead] {
border: 2px solid rgb(0, 102, 204);
padding: 1px;
}
-.wallpaper-attr {
+#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.
-webkit-box-orient: vertical;
display: -webkit-box;
}
+#wallpaper-attr[hidden] {
flackr 2012/09/13 02:34:27 perhaps just *[hidden]?
bshe 2012/09/13 03:45:48 Done.
+ display: none;
+}
+
.bottom-bar {
-webkit-box-align: center;
-webkit-box-orient: horizontal;
« 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