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

Unified Diff: ui/file_manager/file_manager/foreground/js/ui/directory_tree.js

Issue 884653002: Update visual style of directory tree. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove eject_white.png Created 5 years, 11 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/js/ui/directory_tree.js
diff --git a/ui/file_manager/file_manager/foreground/js/ui/directory_tree.js b/ui/file_manager/file_manager/foreground/js/ui/directory_tree.js
index 64ebdb0cb6a70d28c62d7001e4c14a85de89733a..07da8edbc7dac8c84d956dcbf8167ed9dda02dd5 100644
--- a/ui/file_manager/file_manager/foreground/js/ui/directory_tree.js
+++ b/ui/file_manager/file_manager/foreground/js/ui/directory_tree.js
@@ -40,8 +40,11 @@ Object.freeze(DirectoryItemTreeBaseMethods);
var TREE_ITEM_INNTER_HTML =
'<div class="tree-row">' +
- ' <span class="expand-icon"></span>' +
- ' <span class="icon"></span>' +
+ ' <paper-ripple fit></paper-ripple>' +
+ ' <span class="expand-icon">' +
+ ' <core-icon icon="expand-more" class="expand-icon"></core-icon>' +
+ ' </span>' +
+ ' <core-icon class="icon"></core-icon>' +
' <span class="label entry-name"></span>' +
'</div>' +
'<div class="tree-children"></div>';
@@ -351,7 +354,7 @@ function SubDirectoryItem(label, dirEntry, parentDirItem, tree) {
if (location && location.rootType && location.isRootEntry) {
icon.setAttribute('volume-type-icon', location.rootType);
} else {
- icon.setAttribute('file-type-icon', 'folder');
+ icon.icon = 'folder';
item.updateSharedStatusIcon();
}
@@ -399,7 +402,10 @@ SubDirectoryItem.prototype.updateSharedStatusIcon = function() {
[this.dirEntry_],
'external',
function(metadata) {
- icon.classList.toggle('shared', metadata[0] && metadata[0].shared);
+ if (metadata[0] && metadata[0].shared)
+ icon.icon = 'folder-shared';
+ else
+ icon.icon = 'folder';
});
};
« no previous file with comments | « ui/file_manager/file_manager/foreground/js/main_scripts.js ('k') | ui/file_manager/file_manager/foreground/js/ui/tree.css.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698