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

Unified Diff: chrome/browser/resources/file_manager/css/file_manager.css

Issue 10411018: [FileBrowser] Added DefaultAction dialog to choose default action. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 7 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
Index: chrome/browser/resources/file_manager/css/file_manager.css
diff --git a/chrome/browser/resources/file_manager/css/file_manager.css b/chrome/browser/resources/file_manager/css/file_manager.css
index 5df2f73619946e4ed3ea7b3784f190d91d58d4d8..a15e2ec1a1b2361eaae4ada20fd9ba7580f27e60 100644
--- a/chrome/browser/resources/file_manager/css/file_manager.css
+++ b/chrome/browser/resources/file_manager/css/file_manager.css
@@ -558,33 +558,61 @@ div.img-container > img {
position: absolute;
}
+/* Padding counterweights negative margins of items, thus eliminating scroll
+ bar when it's not needed. Max height is set to fit 8 items before showing
+ scroll bar. */
+#default-actions-list {
+ max-height: 328px;
+ padding: 1px 0;
+}
+
+#default-actions-list > * {
dgozman 2012/05/18 14:50:06 You should use 'li' instead of * here and below.
Dmitry Zvorygin 2012/05/22 14:32:59 Done.
+ display: -webkit-box;
+}
+
+#default-actions-list > * > * {
+ display: block;
+ padding: 0 5px;
+}
+
+#default-actions-list > * > img {
+ height: 16px;
+ padding-top: 12px;
+}
+
+
#list-container list > *,
-#list-container grid > * {
+#list-container grid > *,
+#default-actions-list > * {
background-image: none;
border-radius: 0;
line-height: 30px;
}
#list-container list > [selected],
-#list-container grid > [selected] {
+#list-container grid > [selected],
+#default-actions-list > [selected] {
background-color: rgb(203, 219, 241);
}
-#list-container list > li.table-row[selected] {
+#list-container list > li.table-row[selected],
+#default-actions-list > li[selected] {
border-top: 1px solid rgb(224, 233, 247);
}
#list-container list > .accepts[selected],
#list-container grid > .accepts[selected],
#list-container list > [selected]:hover,
-#list-container grid > [selected]:hover {
+#list-container grid > [selected]:hover,
+#default-actions-list > [selected]:hover {
background-color: rgb(193, 211, 236);
}
#list-container list > .accepts,
#list-container grid > .accepts,
#list-container list > :hover,
-#list-container grid > :hover {
+#list-container grid > :hover,
+#default-actions-list > :hover {
background-color: #f1f1f1;
}
@@ -832,7 +860,8 @@ input.rename {
opacity: 1 !important;
}
-#list-container li.table-row {
+#list-container li.table-row,
+#default-actions-list li {
border: none;
border-top: 1px solid transparent;
line-height: 39px;

Powered by Google App Engine
This is Rietveld 408576698