Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright 2016 The Chromium Authors. All rights reserved. | 1 // Copyright 2016 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-side-bar', | 6 is: 'history-side-bar', |
| 7 | 7 |
| 8 properties: { | 8 properties: { |
| 9 selectedPage: { | 9 selectedPage: { |
| 10 type: String, | 10 type: String, |
| 11 notify: true | 11 notify: true |
| 12 } | 12 }, |
| 13 }, | |
| 14 | |
| 15 toggle: function() { | |
| 16 this.$.drawer.toggle(); | |
| 17 }, | |
| 18 | |
| 19 /** @private */ | |
| 20 onDrawerFocus_: function() { | |
| 21 // app-drawer will focus its first focusable child when it opens. Instead, | |
| 22 // make it focus itself (with tabindex=0), and then immediately delegate | |
| 23 // focus to the selected item. | |
|
calamity
2016/06/17 03:02:00
I don't fully understand this comment. What proble
tsergeant
2016/06/20 05:05:57
I've expanded the comment to try and explain it. L
| |
| 24 this.$.menu.selectedItem.focus(); | |
| 25 }, | |
| 26 | |
| 27 /** @private */ | |
| 28 onSelectorActivate_: function() { | |
| 29 this.$.drawer.close(); | |
| 13 }, | 30 }, |
| 14 | 31 |
| 15 /** | 32 /** |
| 16 * Handles menu selection changes. | |
| 17 * @param {Event} e | |
| 18 * @private | |
| 19 */ | |
| 20 onSelect_: function(e) { | |
| 21 this.fire('unselect-all'); | |
| 22 this.selectedPage = e.detail.item.getAttribute('view-id'); | |
| 23 }, | |
| 24 | |
| 25 /** | |
| 26 * Relocates the user to the clear browsing data section of the settings page. | 33 * Relocates the user to the clear browsing data section of the settings page. |
| 27 * @private | 34 * @private |
| 28 */ | 35 */ |
| 29 onClearBrowsingDataTap_: function() { | 36 onClearBrowsingDataTap_: function() { |
| 30 window.location.href = 'chrome://settings/clearBrowserData'; | 37 window.location.href = 'chrome://settings/clearBrowserData'; |
| 31 }, | 38 }, |
| 32 }); | 39 }); |
| OLD | NEW |