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

Side by Side Diff: chrome/browser/resources/md_history/history_list.js

Issue 2334553002: [MD History] Add keyboard navigation to the main history list. (Closed)
Patch Set: address comments Created 4 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 Polymer({ 5 Polymer({
6 is: 'history-list', 6 is: 'history-list',
7 7
8 behaviors: [HistoryListBehavior], 8 behaviors: [HistoryListBehavior],
9 9
10 properties: { 10 properties: {
11 // The search term for the current query. Set when the query returns. 11 // The search term for the current query. Set when the query returns.
12 searchedTerm: { 12 searchedTerm: {
13 type: String, 13 type: String,
14 value: '', 14 value: '',
15 }, 15 },
16 16
17 querying: Boolean, 17 querying: Boolean,
18 18
19 // An array of history entries in reverse chronological order. 19 // An array of history entries in reverse chronological order.
20 historyData_: Array, 20 historyData_: Array,
21 21
22 resultLoadingDisabled_: { 22 resultLoadingDisabled_: {
23 type: Boolean, 23 type: Boolean,
24 value: false, 24 value: false,
25 }, 25 },
26
27 lastFocused_: Object,
26 }, 28 },
27 29
28 listeners: { 30 listeners: {
29 'scroll': 'notifyListScroll_', 31 'scroll': 'notifyListScroll_',
30 'remove-bookmark-stars': 'removeBookmarkStars_', 32 'remove-bookmark-stars': 'removeBookmarkStars_',
31 }, 33 },
32 34
33 /** @override */ 35 /** @override */
34 attached: function() { 36 attached: function() {
35 // It is possible (eg, when middle clicking the reload button) for all other 37 // It is possible (eg, when middle clicking the reload button) for all other
(...skipping 134 matching lines...) Expand 10 before | Expand all | Expand 10 after
170 172
171 /** 173 /**
172 * @param {number} index 174 * @param {number} index
173 * @return {string} 175 * @return {string}
174 * @private 176 * @private
175 */ 177 */
176 pathForItem_: function(index) { 178 pathForItem_: function(index) {
177 return 'historyData_.' + index; 179 return 'historyData_.' + index;
178 }, 180 },
179 }); 181 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698