From fd8bac20031a5f1aaba018bd5a17f9a955846346 Mon Sep 17 00:00:00 2001 From: lilia Date: Fri, 4 Mar 2016 06:23:03 -0800 Subject: [PATCH] Fix conversation list ordering Order conversation list by timestamp instead of active_at. The former is the send time of the most recent message. The latter is typically the receive time of that message. This can cause mis-ordering if you send a message while processing a backlog of incoming messages. Fixes #617 // FREEBIE --- js/conversation_controller.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/js/conversation_controller.js b/js/conversation_controller.js index 8602557d..31988245 100644 --- a/js/conversation_controller.js +++ b/js/conversation_controller.js @@ -18,7 +18,7 @@ this.listenTo(conversations, 'add change:active_at', this.addActive); }, comparator: function(model) { - return -model.get('active_at'); + return -model.get('timestamp'); }, addActive: function(model) { if (model.get('active_at')) {