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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/resources/ntp_search/new_tab.js

Issue 11412214: NTP5: Fine tuning of Apps implementation. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: sync and rebase Created 8 years, 1 month 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
« chrome/browser/resources/ntp_search/apps_page.js ('K') | « chrome/browser/resources/ntp_search/mock/debug.css ('k') | chrome/browser/resources/ntp_search/thumbnail_page.js » ('j') | chrome/browser/resources/ntp_search/tile_page.css » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/ntp_search/new_tab.js
diff --git a/chrome/browser/resources/ntp_search/new_tab.js b/chrome/browser/resources/ntp_search/new_tab.js
index 4a83ddb3898d3e0d1faaa3ace6d65af430a909cd..c8156d2ff519301c1b2fc295ed8e864d4dd5e95e 100644
--- a/chrome/browser/resources/ntp_search/new_tab.js
+++ b/chrome/browser/resources/ntp_search/new_tab.js
@@ -422,8 +422,8 @@ cr.define('ntp', function() {
// Sort by launch ordinal
Evan Stade 2012/11/30 20:52:46 update
pedro (no code reviews) 2012/11/30 22:01:18 Done.
apps.sort(function(a, b) {
- return a.app_launch_ordinal > b.app_launch_ordinal ? 1 :
- a.app_launch_ordinal < b.app_launch_ordinal ? -1 : 0;
+ return a.title.toLocaleLowerCase() > b.title.toLocaleLowerCase() ? 1 :
+ a.title.toLocaleLowerCase() < b.title.toLocaleLowerCase() ? -1 : 0;
});
// An app to animate (in case it was just installed).
« chrome/browser/resources/ntp_search/apps_page.js ('K') | « chrome/browser/resources/ntp_search/mock/debug.css ('k') | chrome/browser/resources/ntp_search/thumbnail_page.js » ('j') | chrome/browser/resources/ntp_search/tile_page.css » ('J')

Powered by Google App Engine
This is Rietveld 408576698