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

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: Merge to trunk 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 868a4e2fd3d3a23811c3972d227d1b297691f514..16469269f27bdb6d1fb06a0c97fde794c3ef136a 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, an incorrect
+ * image size may be used during layout. Some images may be misaligned as a
+ * 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;
}
@@ -151,6 +156,11 @@ div.sidebar-splitter {
width: 128px;
}
+/* [hidden] does display:none, but its priority is too low. */
+[hidden] {
+ display: none !important;
+}
+
/* The butter bar styles are copied from file_manager.css. We will revisit
* it to see if we can share some code after butter bar is integrated with
* Photo Editor.
« 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