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

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: address reviews 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..550643134e329bcc7ddb34f7f2da1fe40c30b229 100644
--- a/chrome/browser/resources/chromeos/wallpaper_manager/css/wallpaper_manager.css
+++ b/chrome/browser/resources/chromeos/wallpaper_manager/css/wallpaper_manager.css
@@ -82,11 +82,6 @@ div.sidebar-splitter {
display: -webkit-box;
}
-#category-container[hidden],
-#custom-container[hidden] {
- display: none;
-}
-
#custom-container {
background-color: #FFF;
margin: 10px;
@@ -121,12 +116,22 @@ 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
flackr 2012/09/13 20:55:01 grammar nit: "Otherwise, an incorrect image size m
bshe 2012/09/13 21:13:54 Done.
+ * result. See http://code.google.com/p/chromium/issues/detail?id=148480 for
+ * screen shot.
+ */
+.image-picker img {
+ height: 80px;
+ width: 128px;
+}
+
.image-picker [selected][lead] {
border: 2px solid rgb(0, 102, 204);
padding: 1px;
}
-.wallpaper-attr {
+#wallpaper-attribute {
-webkit-box-orient: vertical;
display: -webkit-box;
}
@@ -150,3 +155,8 @@ div.sidebar-splitter {
height: 80px;
width: 128px;
}
+
+/* [hidden] does display:none, but its priority is too low. */
+[hidden] {
+ display: none !important;
+}
« 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