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

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

Issue 2020963002: MD History: Add responsive layout which hides the sidebar in thin windows (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Cleanup and closure Created 4 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 unified diff | Download patch
OLDNEW
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 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698