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

Unified Diff: chrome/browser/resources/ntp4/recently_closed.js

Issue 12038090: Layout the NTP footer with flexboxes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 11 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/ntp4/recently_closed.js
diff --git a/chrome/browser/resources/ntp4/recently_closed.js b/chrome/browser/resources/ntp4/recently_closed.js
index 8dfdf06d57bbacdfc913ec03b3403acc252807d4..e9071892f7caf88940c13522961f197239b8e7d5 100644
--- a/chrome/browser/resources/ntp4/recently_closed.js
+++ b/chrome/browser/resources/ntp4/recently_closed.js
@@ -36,7 +36,6 @@ cr.define('ntp', function() {
document.body.appendChild(this.menu);
this.needsRebuild_ = true;
- this.hidden = true;
this.anchorType = cr.ui.AnchorType.ABOVE;
this.invertLeftRight = true;
},
@@ -64,7 +63,10 @@ cr.define('ntp', function() {
set dataItems(dataItems) {
this.dataItems_ = dataItems;
this.needsRebuild_ = true;
- this.hidden = !dataItems.length;
+ if (dataItems.length)
Dan Beam 2013/01/31 20:22:11 same nit re: this.classList.toggle()
dconnelly 2013/02/04 16:40:45 Same comment as other_sessions.js--that form of to
+ this.classList.remove('invisible');
+ else
+ this.classList.add('invisible');
},
/**

Powered by Google App Engine
This is Rietveld 408576698