Merge pull request sj26#78 from asynchrony/master

Sort messages from newest to oldest [also closes sj26#70].
apeiros · Feb 10, 2013 · c971f54 · c971f54
2 parents fbd6b94 + 2ae395a
commit c971f54
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 11 deletions.
diff --git a/lib/mail_catcher/mail.rb b/lib/mail_catcher/mail.rb
@@ -69,7 +69,7 @@ def latest_created_at
   end
 
   def messages
-    @messages_query ||= db.prepare "SELECT id, sender, recipients, subject, size, created_at FROM message ORDER BY created_at ASC"
+    @messages_query ||= db.prepare "SELECT id, sender, recipients, subject, size, created_at FROM message ORDER BY created_at DESC"
     @messages_query.execute.map do |row|
       Hash[row.fields.zip(row)].tap do |message|
         message["recipients"] &&= ActiveSupport::JSON.decode message["recipients"]

diff --git a/public/javascripts/application.coffee b/public/javascripts/application.coffee
@@ -173,7 +173,7 @@ class MailCatcher
     $('#messages tbody tr').show()
 
   addMessage: (message) ->
-    $('#messages tbody').append \
+    $('#messages tbody').prepend \
       $('<tr />').attr('data-message-id', message.id.toString())
         .append($('<td/>').text(message.sender or "No sender").toggleClass("blank", !message.sender))
         .append($('<td/>').text((message.recipients || []).join(', ') or "No receipients").toggleClass("blank", !message.recipients.length))

diff --git a/public/javascripts/application.js b/public/javascripts/application.js