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

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

Issue 914353004: Update import details panel to comply (mostly) with UX direction. (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
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 04099eb53395b6c378eaa9ddcc4573868c031ee7..dcfbf3a1e8bbcf4e7153f84cb46a8bdbc144b551 100644
--- a/ui/file_manager/file_manager/foreground/css/file_manager.css
+++ b/ui/file_manager/file_manager/foreground/css/file_manager.css
@@ -349,25 +349,72 @@ body.selecting .dialog-header {
#cloud-import-details {
background-color: white;
- height: 300px;
- opacity: .85;
- padding: 10px;
+ border-radius: 4px;
+ box-shadow: 2px 2px 6px #555555;
+ font-family: 'Roboto';
mtomasz 2015/02/12 03:22:01 nit: Roboto should be default in the entire Files
Steve McKay 2015/02/12 23:15:53 Done.
position: absolute;
- right: 0px;
- top: 49px;
+ right: 75px;
+ top: 38px;
mtomasz 2015/02/12 03:22:01 Please comment all magical numbers. Does it have t
Steve McKay 2015/02/12 23:15:53 Done.
transition: all 0.2s ease;
mtomasz 2015/02/12 03:22:01 Position transition with top/left/right/bottom are
Steve McKay 2015/02/12 23:15:53 Done.
width: 250px;
z-index: 5;
}
-#cloud-import-details.offscreen {
+#cloud-import-details.hidden {
+ right: 50px;
opacity: 0;
- right: -202px;
}
-#cloud-import-details div {
- border: 0px 0px 1px 0px solid gray;
- padding: 10px;
+#cloud-import-details .banner {
+ background-color: #37474F;
+ border-radius: 4px 4px 0px 0px ;
mtomasz 2015/02/12 03:22:01 nit: \s; -> ;
Steve McKay 2015/02/12 23:15:53 Done.
+ color: white;
+ padding: 20px;
+ font-size: 12pt;
mtomasz 2015/02/12 03:22:01 nit: Please use percents for font sizes, so the UI
Steve McKay 2015/02/12 23:15:53 K. Done.
+ line-height: 1.2em;
+}
+
+#cloud-import-details .banner .title {
+ font-size: 14pt;
+ margin-bottom: 5px;
+}
+
+#cloud-import-details .main {
+ display: flex;
+ flex-direction: column;
+ padding: 20px;
+}
+
+#cloud-import-details .status {
+ display: flex;
+ flex-direction: row;
+ font-color: #646464;
+ font-size: 12pt;
+}
+
+#cloud-import-details .status .leader {
+ font-color: #333333;
+ font-size: 13pt;
+}
+
+#cloud-import-details .status core-icon {
+ display: block;
+ height: 16px;
+ margin-right: 10px;
+ opacity: .6;
+ width: 16px;
+}
+
+#cloud-import-details .status .destination-link {
+ color: #1BA8F3;
+ cursor: pointer;
+}
+
+#cloud-import-details paper-button.import {
+ align-self: flex-end;
+ background-color: #1BA8F3;
+ color: white;
+ margin-top: 20px;
}
#files-selected-label {

Powered by Google App Engine
This is Rietveld 408576698