moparisthebest
/
mailcatcher
mirror of https://github.com/moparisthebest/mailcatcher
1
0
Fork 0
Code Issues Releases Wiki Activity

Fix attachments not appearing after clearing

This commit is contained in:
Samuel Cochran 2011-07-08 14:47:41 +08:00
parent 6b5f8cbfca
commit e9da024345
3 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
public/javascripts/application.coffee
View File

@ -101,9 +101,10 @@ class MailCatcher
$("#message .views .tab:visible:first").addClass "selected" $("#message .views .tab:visible:first").addClass "selected"
if message.attachments.length if message.attachments.length
$('#message .metadata dd.attachments ul').empty() $ul = $('<ul/>').appendTo($('#message .metadata dd.attachments').empty())
$.each message.attachments, (i, attachment) -> $.each message.attachments, (i, attachment) ->
$('#message .metadata dd.attachments ul').append($('<li>').append($('<a>').attr('href', attachment['href']).addClass(attachment['type'].split('/', 1)[0]).addClass(attachment['type'].replace('/', '-')).text(attachment['filename']))); $ul.append($('<li>').append($('<a>').attr('href', attachment['href']).addClass(attachment['type'].split('/', 1)[0]).addClass(attachment['type'].replace('/', '-')).text(attachment['filename'])));
$('#message .metadata .attachments').show() $('#message .metadata .attachments').show()
else else
$('#message .metadata .attachments').hide() $('#message .metadata .attachments').hide()

5
public/javascripts/application.js
View File

@ -99,6 +99,7 @@
$('#messages tbody tr:not([data-message-id="' + id + '"])').removeClass('selected'); $('#messages tbody tr:not([data-message-id="' + id + '"])').removeClass('selected');
$('#messages tbody tr[data-message-id="' + id + '"]').addClass('selected'); $('#messages tbody tr[data-message-id="' + id + '"]').addClass('selected');
return $.getJSON('/messages/' + id + '.json', __bind(function(message) { return $.getJSON('/messages/' + id + '.json', __bind(function(message) {
var $ul;
$('#message .metadata dd.created_at').text(this.formatDate(message.created_at)); $('#message .metadata dd.created_at').text(this.formatDate(message.created_at));
$('#message .metadata dd.from').text(message.sender); $('#message .metadata dd.from').text(message.sender);
$('#message .metadata dd.to').text((message.recipients || []).join(', ')); $('#message .metadata dd.to').text((message.recipients || []).join(', '));
@ -119,9 +120,9 @@
$("#message .views .tab:visible:first").addClass("selected"); $("#message .views .tab:visible:first").addClass("selected");
} }
if (message.attachments.length) { if (message.attachments.length) {
$('#message .metadata dd.attachments ul').empty(); $ul = $('<ul/>').appendTo($('#message .metadata dd.attachments').empty());
$.each(message.attachments, function(i, attachment) { $.each(message.attachments, function(i, attachment) {
return $('#message .metadata dd.attachments ul').append($('<li>').append($('<a>').attr('href', attachment['href']).addClass(attachment['type'].split('/', 1)[0]).addClass(attachment['type'].replace('/', '-')).text(attachment['filename']))); return $ul.append($('<li>').append($('<a>').attr('href', attachment['href']).addClass(attachment['type'].split('/', 1)[0]).addClass(attachment['type'].replace('/', '-')).text(attachment['filename'])));
}); });
$('#message .metadata .attachments').show(); $('#message .metadata .attachments').show();
} else { } else {

1
views/index.haml
View File

@ -41,7 +41,6 @@
%dd.subject %dd.subject
%dt.attachments Attachments %dt.attachments Attachments
%dd.attachments %dd.attachments
%ul
%nav.views %nav.views
%ul %ul
%li.format.tab.html.selected{'data-message-format' => 'html'} %li.format.tab.html.selected{'data-message-format' => 'html'}