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

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

Issue 2920413002: MD Bookmarks: Focus sidebar nodes on click without showing an outline (Closed)
Patch Set: Rewrite to use a function rather than a property Created 3 years, 6 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/folder_node.js
diff --git a/chrome/browser/resources/md_bookmarks/folder_node.js b/chrome/browser/resources/md_bookmarks/folder_node.js
index 09dd8099ee59984910986f240fbdfa31b6a09430..b959dca03851690f155cd9a5a631281a0cad5f9c 100644
--- a/chrome/browser/resources/md_bookmarks/folder_node.js
+++ b/chrome/browser/resources/md_bookmarks/folder_node.js
@@ -6,6 +6,7 @@ Polymer({
is: 'bookmarks-folder-node',
behaviors: [
+ bookmarks.MouseFocusBehavior,
bookmarks.StoreClient,
],
@@ -69,7 +70,7 @@ Polymer({
}
},
- /** @return {HTMLElement} */
+ /** @return {!HTMLElement} */
calamity 2017/06/08 03:34:18 @override
tsergeant 2017/06/08 05:03:40 Done.
getFocusTarget: function() {
return this.$.container;
},
@@ -339,6 +340,6 @@ Polymer({
* @return {string}
*/
getTabIndex_: function() {
- return this.isSelectedFolder_ ? '0' : '';
+ return this.isSelectedFolder_ ? '0' : '-1';
},
});

Powered by Google App Engine
This is Rietveld 408576698