Dressed up a few more messages.

email_confirm, message_notification and signup_confirm now use the new templates. This fixes some, but not all, of the currently broken tests.

diff --git a/app/models/notifier.rb b/app/models/notifier.rb
index ee40a0d6efad9c48f1c9d242669df674485ce467..8f34866266f5ea7a491c57e4d003877da334df26 100644 (file)
--- a/app/models/notifier.rb
+++ b/app/models/notifier.rb
@@ -77,6 +77,9 @@ class Notifier < ActionMailer::Base
       @replyurl = url_for(:host => SERVER_URL,
                           :controller => "message", :action => "reply",
                           :message_id => message.id)
+      @user_message_author = @from_user
+
+      attach_user_avatar(message.sender)
 
       mail :from => from_address(message.sender.display_name, "m", message.id, message.digest),
            :to => message.recipient.email,

diff --git a/app/views/notifier/email_confirm.html.erb b/app/views/notifier/email_confirm.html.erb
index 5b7c74d7f52dd608bcb41569b6db3c5577d6654b..5c639d57fdf42a2222c00fc7aaebe28fdc96af8d 100644 (file)
--- a/app/views/notifier/email_confirm.html.erb
+++ b/app/views/notifier/email_confirm.html.erb
@@ -1,7 +1,9 @@
-<p><%= t 'notifier.email_confirm_html.greeting' %></p>
+<% content_for :body do %>
+  <p style="margin: 0"><%= t 'notifier.email_confirm_html.greeting' %></p>
 
-<p><%= t 'notifier.email_confirm_html.hopefully_you', :server_url => SERVER_URL, :new_address => @address %></p>
+  <p><%= t 'notifier.email_confirm_html.hopefully_you', :server_url => SERVER_URL, :new_address => @address %></p>
 
-<p><%= t 'notifier.email_confirm_html.click_the_link' %></p>
+  <p><%= t 'notifier.email_confirm_html.click_the_link' %></p>
 
-<p><a href="<%= @url %>"><%= @url %></a></p>
+  <p><a href="<%= @url %>" style="white-space: nowrap"><%= @url %></a></p>
+<% end %>

diff --git a/app/views/notifier/message_notification.html.erb b/app/views/notifier/message_notification.html.erb
index 19704251d8041e309250888c63afe0ef933cee44..823f1487aa86f442c68a40640ceb23fb6643fa00 100644 (file)
--- a/app/views/notifier/message_notification.html.erb
+++ b/app/views/notifier/message_notification.html.erb
@@ -1,9 +1,24 @@
-<p><%= t'notifier.message_notification.hi', :to_user => @to_user %></p>
+<% content_for :body do %>
+  <p style="margin: 0">
+    <%= t'notifier.message_notification.hi', :to_user => @to_user %>
+  </p>
+  <p>
+    <%= raw t'notifier.message_notification.header',
+            :from_user => link_to_user(@from_user),
+            :subject => content_tag("em", @title)
+    %>
+  </p>
 
-<p><%= raw t'notifier.message_notification.header', :from_user => link_to(@from_user, :host => SERVER_URL, :controller => :user, :action => :view, :display_name => @from_user), :subject => @title %></p>
+  <%= render "notifier/user_message_table", :captured => capture { %>
+    <%= @text.to_html %>
+  <% } %>
+<% end %>
 
-==
-<%= @text.to_html %>
-==
-
-<p><%= t'notifier.message_notification.footer_html', :readurl => link_to(@readurl, @readurl), :replyurl => link_to(@replyurl, @replyurl) %></p>
+<% content_for :footer do %>
+  <p>
+    <%= t'notifier.message_notification.footer_html',
+          :readurl => link_to(@readurl, @readurl) + tag(:br),
+          :replyurl => link_to(@replyurl, @replyurl)
+    %>
+  </p>
+<% end %>

diff --git a/app/views/notifier/signup_confirm.html.erb b/app/views/notifier/signup_confirm.html.erb
index 814deee9171220d93896643b05484d1bb9133076..66528bd8345626b448044c51cfe622e93a758c3a 100644 (file)
--- a/app/views/notifier/signup_confirm.html.erb
+++ b/app/views/notifier/signup_confirm.html.erb
@@ -1,9 +1,11 @@
-<p><%= t("notifier.signup_confirm.greeting") %></p>
+<% content_for :body do %>
+  <p style="margin: 0"><%= t("notifier.signup_confirm.greeting") %></p>
 
-<p><%= t("notifier.signup_confirm.created", :site_url => SERVER_URL) %></p>
+  <p><%= t("notifier.signup_confirm.created", :site_url => SERVER_URL) %></p>
 
-<p><%= t("notifier.signup_confirm.confirm") %></p>
+  <p><%= t("notifier.signup_confirm.confirm") %></p>
 
-<p><%= link_to @url, @url %></p>
+  <p><%= link_to @url, @url, :style => "white-space: nowrap" %></nobr></p>
 
-<p><%= t("notifier.signup_confirm.welcome") %></p>
+  <p><%= t("notifier.signup_confirm.welcome") %></p>
+<% end %>