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

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

Issue 7776001: ntp4: improved app install, try 2 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix trash change Created 9 years, 4 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/apps_page.js
diff --git a/chrome/browser/resources/ntp4/apps_page.js b/chrome/browser/resources/ntp4/apps_page.js
index a81c34399f8767ee419b1e6af00010164b45ac51..904bf56589d5d1c6f866ff4648077f0f94fa2200 100644
--- a/chrome/browser/resources/ntp4/apps_page.js
+++ b/chrome/browser/resources/ntp4/apps_page.js
@@ -412,6 +412,13 @@ cr.define('ntp4', function() {
* @param {?boolean} animate If true, the app tile plays an animation.
*/
appendApp: function(appData, animate) {
+ if (animate) {
+ // Select the page and scroll all the way down so the animation is
+ // visible.
+ ntp4.getCardSlider().selectCardByValue(this);
+ console.log('selecting card');
Rick Byers 2011/08/29 16:01:00 We don't generally want to leave diagnostic loggin
Evan Stade 2011/08/29 18:15:16 correct, left here by mistake
+ this.content_.scrollTop = this.content_.scrollHeight;
+ }
this.appendTile(new App(appData), animate);
},

Powered by Google App Engine
This is Rietveld 408576698