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

Unified Diff: chrome/browser/resources/md_bookmarks/list.js

Issue 2614703003: [MD Bookmarks] Add search. (Closed)
Patch Set: Created 3 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: chrome/browser/resources/md_bookmarks/list.js
diff --git a/chrome/browser/resources/md_bookmarks/list.js b/chrome/browser/resources/md_bookmarks/list.js
index 426cbb33be433e5dafe803e9ac440ab97314edc7..ae3bcc24a15ec24b6c65cc2d591e6aaf9cf30ecb 100644
--- a/chrome/browser/resources/md_bookmarks/list.js
+++ b/chrome/browser/resources/md_bookmarks/list.js
@@ -8,12 +8,24 @@ Polymer({
properties: {
/** @type {BookmarkTreeNode} */
selectedNode: Object,
+
+ searchTerm: String,
tsergeant 2017/01/06 02:43:54 Is it possible to make it so that the list doesn't
angelayang 2017/01/12 05:04:32 That makes a lot of sense yep
+
+ searchResult: {
+ type: Array,
+ },
+
+ displayedList_: Array,
},
listeners: {
'toggle-menu': 'onToggleMenu_'
},
+ observers: [
+ 'updateDisplayList_(selectedNode.children, searchResult)',
+ ],
+
/**
* @param {Event} e
* @private
@@ -45,5 +57,15 @@ Polymer({
var menu = /** @type {!CrActionMenuElement} */ (
this.$.dropdown);
menu.close();
- }
+ },
+
+ /** @private */
+ updateDisplayList_: function() {
+ if (this.searchTerm == '') {
+ this.displayedList_ =
+ /** @type {!Array<!BookmarkTreeNode>} */ (this.selectedNode.children);
+ } else {
+ this.displayedList_ = this.searchResult;
+ }
+ },
});

Powered by Google App Engine
This is Rietveld 408576698