Chromium Code Reviews| Index: ui/webui/resources/cr_elements/cr_toolbar/cr_toolbar_search_field.js |
| diff --git a/ui/webui/resources/cr_elements/cr_toolbar/cr_toolbar_search_field.js b/ui/webui/resources/cr_elements/cr_toolbar/cr_toolbar_search_field.js |
| index aac426820c43e00e97956025673933bea7ae198f..2d60fd9b3926346bf133da3882e77abca05f7e7d 100644 |
| --- a/ui/webui/resources/cr_elements/cr_toolbar/cr_toolbar_search_field.js |
| +++ b/ui/webui/resources/cr_elements/cr_toolbar/cr_toolbar_search_field.js |
| @@ -25,6 +25,15 @@ Polymer({ |
| 'tap': 'showSearch_', |
| }, |
| + /** |
| + * @param {boolean} narrow |
| + * @return {number} |
| + * @private |
| + */ |
| + computeIconTabIndex_: function(narrow) { |
| + return narrow ? 0 : -1; |
| + }, |
| + |
| /** @private */ |
| onInputBlur_: function() { |
| if (!this.hasSearchText) |
| @@ -32,6 +41,16 @@ Polymer({ |
| }, |
| /** |
| + * Expand the search field when a key is pressed with it focused. This ensures |
| + * it can be used correctly by tab-focusing. We use keypress instead of |
|
dpapad
2016/06/01 01:27:10
Thanks for adding the explanatory comment, it help
tsergeant
2016/06/01 01:46:35
Done.
|
| + * keydown to avoid expanding on non-text keys (shift, escape, etc). |
| + * @private |
| + */ |
| + onSearchTermKeypress_: function() { |
| + this.showingSearch = true; |
| + }, |
| + |
| + /** |
| * @param {Event} e |
| * @private |
| */ |