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

Unified Diff: chrome/browser/resources/new_new_tab.js

Issue 6297013: [NTP] Allow reordering of apps via drag and drop. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: nit 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
« no previous file with comments | « chrome/browser/resources/new_new_tab.html ('k') | chrome/browser/resources/ntp/apps.css » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/new_new_tab.js
diff --git a/chrome/browser/resources/new_new_tab.js b/chrome/browser/resources/new_new_tab.js
index 877144c78cfd8970ca3bf5484dd37e19e1012a1c..fdc1c61f37afc27c6cfe708aa40b8d6d6ac81a8e 100644
--- a/chrome/browser/resources/new_new_tab.js
+++ b/chrome/browser/resources/new_new_tab.js
@@ -187,7 +187,7 @@ function createForeignSession(client, name) {
// Sort tabs by MRU order
win.tabs.sort(function(a, b) {
return a.timestamp < b.timestamp;
- })
+ });
// Create individual tab information.
win.tabs.forEach(function(data) {
@@ -205,7 +205,7 @@ function createForeignSession(client, name) {
handleIfEnterKey(maybeOpenForeignTab));
winSpan.appendChild(tabEl);
- })
+ });
// Append the window.
stack.appendChild(winSpan);
@@ -309,6 +309,7 @@ function handleWindowResize() {
if (layoutMode != oldLayoutMode){
mostVisited.useSmallGrid = b;
mostVisited.layout();
+ apps.layout({force:true});
renderRecentlyClosed();
renderForeignSessions();
updateAllMiniviewClippings();
@@ -626,6 +627,10 @@ function showSection(section) {
mostVisited.visible = true;
mostVisited.layout();
break;
+ case Section.APPS:
+ apps.visible = true;
+ apps.layout({disableAnimations:true});
+ break;
}
}
}
@@ -650,6 +655,10 @@ function hideSection(section) {
mostVisited.visible = false;
mostVisited.layout();
break;
+ case Section.APPS:
+ apps.visible = false;
+ apps.layout();
+ break;
}
var el = getSectionElement(section);
« no previous file with comments | « chrome/browser/resources/new_new_tab.html ('k') | chrome/browser/resources/ntp/apps.css » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698