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

Unified Diff: ui/file_manager/file_manager/foreground/css/file_manager.css

Issue 940933002: Files.app: Re-design auto-complete suggestion list. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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 | ui/file_manager/file_manager/foreground/js/ui/search_box.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/file_manager/file_manager/foreground/css/file_manager.css
diff --git a/ui/file_manager/file_manager/foreground/css/file_manager.css b/ui/file_manager/file_manager/foreground/css/file_manager.css
index 439720e2ea1d8eb6a5ecfaac75685d8bc92ef22c..5e1bed00e0cdd92bd405f6c247aa497e4c5031d7 100644
--- a/ui/file_manager/file_manager/foreground/css/file_manager.css
+++ b/ui/file_manager/file_manager/foreground/css/file_manager.css
@@ -578,12 +578,18 @@ body.selecting #search-box {
}
#search-box paper-input-decorator /deep/ .focused-underline {
- background-color: rgba(255, 255, 255, 0.5);
+ background-color: white;
+ height: 1px;
+}
+
+#search-box.has-cursor,
+#search-box.has-text {
+ -webkit-margin-end: 12px;
}
#search-box.has-cursor input,
#search-box.has-text input {
- width: 176px;
+ width: 218px;
}
#search-box .clear {
@@ -1606,34 +1612,35 @@ body[drive='error'] #unmounted-panel > .error,
}
list.autocomplete-suggestions {
- -webkit-margin-start: -38px;
- background-color: rgb(250, 250, 250);
+ -webkit-margin-start: -36px;
+ background-color: white;
border-radius: 3px;
box-shadow: 0 1px 4px 0 rgba(0, 0, 0, .5);
box-sizing: border-box; /* To match the width with the search box's. */
- color: rgb(34, 34, 34);
+ color: rgb(90, 90, 90);
flex: none;
margin-top: 10px;
overflow: hidden;
- padding: 5px 0;
+ padding: 0;
position: fixed;
- width: 300px !important; /* This overrides the value specified by script. */
+ width: 260px !important; /* This overrides the value specified by script. */
z-index: 550;
}
list.autocomplete-suggestions > li {
align-items: center;
display: flex;
- padding: 3px 0;
+ height: 40px;
}
list.autocomplete-suggestions > li > div.detail-icon {
- -webkit-margin-end: 6px;
- -webkit-margin-start: 6px;
+ margin: 0 3px;
hirono 2015/02/19 14:25:28 Is it OK to stop to use margin-start/end ?
fukino 2015/02/19 14:27:35 margin-start and margin-end has the same value, so
hirono 2015/02/19 14:29:29 Oops, yes. I misunderstood the meaning of values.
+ flex: none;
}
list.autocomplete-suggestions > li > div.detail-text {
flex: auto;
+ margin: 0 4px;
hirono 2015/02/19 14:25:27 Should we use margin-start/end ?
fukino 2015/02/19 14:27:35 Same with above comment.
overflow-x: hidden;
text-overflow: ellipsis;
}
@@ -1645,26 +1652,13 @@ list.autocomplete-suggestions > li > div.detail-text em {
list.autocomplete-suggestions > li > div[search-icon] {
background: -webkit-image-set(
- url(../images/files/ui/search_icon_active.png) 1x,
- url(../images/files/ui/2x/search_icon_active.png) 2x) center no-repeat;
-}
-
-list.autocomplete-suggestions > li[selected] > div[search-icon],
-list.autocomplete-suggestions > li[lead] > div[search-icon] {
- background: -webkit-image-set(
- url(../images/files/ui/search_icon_white.png) 1x,
- url(../images/files/ui/2x/search_icon_white.png) 2x) center no-repeat;
+ url(../images/files/ui/search.png) 1x,
+ url(../images/files/ui/2x/search.png) 2x) center no-repeat;
}
list.autocomplete-suggestions > [selected],
list.autocomplete-suggestions > [lead] {
- background-color: rgb(66, 129, 244);
- color: white;
-}
-
-list.autocomplete-suggestions > [selected] > div.detail-text em,
-list.autocomplete-suggestions > [lead] > div.detail-text em {
- color: white;
+ background-color: rgb(222, 243, 254);
}
#gear-menu {
« no previous file with comments | « no previous file | ui/file_manager/file_manager/foreground/js/ui/search_box.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698