thegamer/Cable-Desktop
2
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Fix conversation list ordering

Browse source 
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
This commit is contained in:
lilia 2016-03-04 06:23:03 -08:00
parent 9fd4760438
commit fd8bac2003
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
js/conversation_controller.js
View file

@ -18,7 +18,7 @@
this.listenTo(conversations, 'add change:active_at', this.addActive); this.listenTo(conversations, 'add change:active_at', this.addActive);
}, },
comparator: function(model) { comparator: function(model) {
return -model.get('active_at'); return -model.get('timestamp');
}, },
addActive: function(model) { addActive: function(model) {
if (model.get('active_at')) { if (model.get('active_at')) {