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

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: Fixed comments. 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 84b19751a26966c155561cb5a61093348b3da2fc..7eef763cd876bcd28647b86ba2397f5059c07361 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 > li {
+ display: -webkit-box;
+}
+
+#default-actions-list > li > * {
+ display: block;
+ padding: 0 5px;
+}
+
+#default-actions-list > li > 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