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

Unified Diff: chrome/browser/resources/md_history/history_toolbar.js

Issue 2165903003: MD History: Show sidebar next to content on wide screens (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@history_fix_footer_link
Patch Set: Add test Created 4 years, 5 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_history/history_toolbar.js
diff --git a/chrome/browser/resources/md_history/history_toolbar.js b/chrome/browser/resources/md_history/history_toolbar.js
index 9562c754ffc3a5706497e956fb8a73d3a0020839..fd6f035afc69c4fd7a5d8db2ac40dd04773de270 100644
--- a/chrome/browser/resources/md_history/history_toolbar.js
+++ b/chrome/browser/resources/md_history/history_toolbar.js
@@ -34,6 +34,12 @@ Polymer({
value: false
},
+ hasDrawer: {
+ type: Boolean,
+ observer: 'hasDrawerChanged_',
+ reflectToAttribute: true,
+ },
+
// Whether domain-grouped history is enabled.
isGroupedMode: {
type: Boolean,
@@ -122,5 +128,10 @@ Polymer({
// TODO(calamity): Fix the format of these dates.
return loadTimeData.getStringF(
'historyInterval', queryStartTime, queryEndTime);
- }
+ },
+
+ /** @private */
+ hasDrawerChanged_: function() {
+ this.updateStyles();
calamity 2016/07/26 05:05:18 Why is this necessary? Shouldn't the attribute ref
tsergeant 2016/07/26 05:59:03 Nope. The polymer custom style shim isn't always a
+ },
});

Powered by Google App Engine
This is Rietveld 408576698