From a9f5513181c7bd78c80602105e1626a972cced7a Mon Sep 17 00:00:00 2001
From: Jessica McKellar <jesstess@humbughq.com>
Date: Fri, 1 Feb 2013 18:07:50 -0500
Subject: [PATCH] subs: compare names using localCompare.

(imported from commit e4dcfe5147601221e720ea6622266402ed09f18a)
---
 zephyr/static/js/subs.js | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/zephyr/static/js/subs.js b/zephyr/static/js/subs.js
index 34e8a60b4f..94ec49cc4d 100644
--- a/zephyr/static/js/subs.js
+++ b/zephyr/static/js/subs.js
@@ -434,14 +434,15 @@ function ajaxSubscribeForCreation(stream, principals, invite_only) {
     });
 }
 
-function people_sort(person1, person2) {
+function people_cmp(person1, person2) {
     // Compares objects of the form used in people_list.
-    if (person1.full_name < person2.full_name) {
+    var name_cmp = person1.full_name.localeCompare(person2.full_name);
+    if (name_cmp < 0) {
         return -1;
-    } else if (person1.full_name > person2.full_name) {
+    } else if (name_cmp > 0) {
         return 1;
     }
-    return person1.email < person2.email ? -1 : 1;
+    return person1.email.localeCompare(person2.email);
 }
 
 function show_new_stream_modal() {
@@ -453,7 +454,7 @@ function show_new_stream_modal() {
     });
 
     $('#people_to_add').html(templates.new_stream_users({
-        users: people_minus_you.sort(people_sort)
+        users: people_minus_you.sort(people_cmp)
     }));
     $('#stream-creation').modal("show");
 }