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

Unified Diff: chrome/browser/resources/ntp/apps.js

Issue 6354016: Fix layout glitches in NTP. Also, lots of naming cleanup. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 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/ntp/apps.js
diff --git a/chrome/browser/resources/ntp/apps.js b/chrome/browser/resources/ntp/apps.js
index 4dd8bc12ce14a1589f4118fd95c06d46691860d4..e61ebe0ea34a85798781e3f0628bddc5e0675a27 100644
--- a/chrome/browser/resources/ntp/apps.js
+++ b/chrome/browser/resources/ntp/apps.js
@@ -66,12 +66,12 @@ function getAppsCallback(data) {
apps.createWebStoreClosedMenuElement());
}
- if (!data.showLauncher || (shownSections & MINIMIZED_APPS)) {
Aaron Boodman 2011/01/22 00:38:41 The second part of the check is no longer needed b
+ if (!data.showLauncher)
appsSection.classList.add('disabled');
- } else {
+ else
appsSection.classList.remove('disabled');
- }
- addClosedMenuFooter(apps.menu, 'apps', MINIMIZED_APPS, Section.APPS);
+
+ addClosedMenuFooter(apps.menu, 'apps', MENU_APPS, Section.APPS);
apps.loaded = true;
if (apps.showPromo)
@@ -121,7 +121,7 @@ var apps = (function() {
function launchApp(appId) {
var appsSection = $('apps');
- var expanded = !appsSection.classList.contains('hidden');
+ var expanded = !appsSection.classList.contains('collapsed');
var element = document.querySelector(
(expanded ? '.maxiview' : '.miniview') + ' a[app-id=' + appId + ']');

Powered by Google App Engine
This is Rietveld 408576698