Use paragraphs instead of line breaks, and avoid using html in translation strings

diff --git a/app/views/confirmations/_resend_success_flash.html.erb b/app/views/confirmations/_resend_success_flash.html.erb
index 1895de761254d3dcd3ed7edb126814f7939474a0..e9763e27618c67f5d111897d8a7e56078e5bccfd 100644 (file)
--- a/app/views/confirmations/_resend_success_flash.html.erb
+++ b/app/views/confirmations/_resend_success_flash.html.erb
@@ -1 +1,2 @@
-<%= t "confirmations.confirm_resend.success_html", :email => email, :sender => sender %>
+<p><%= t ".confirmation_sent", :email => email %></p>
+<p><%= t ".whitelist", :sender => sender %></p>

diff --git a/config/locales/en.yml b/config/locales/en.yml
index 25464bdcfc4e767838f4304df129e35439d88f7a..f6272c918d408ad4b2df95f4b13aa25fc73cb9c3 100644 (file)
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -1535,7 +1535,6 @@ en:
       unknown token: "That confirmation code has expired or does not exist."
       reconfirm_html: "If you need us to resend the confirmation email, <a href=\"%{reconfirm}\">click here</a>."
     confirm_resend:
-      success_html: "We've sent a new confirmation note to %{email} and as soon as you confirm your account you'll be able to get mapping.<br /><br />If you use an antispam system which sends confirmation requests then please make sure you whitelist %{sender} as we are unable to reply to any confirmation requests."
       failure: "User %{name} not found."
     confirm_email:
       heading: Confirm a change of email address
@@ -1544,6 +1543,9 @@ en:
       success: "Confirmed your change of email address!"
       failure: "An email address has already been confirmed with this token."
       unknown_token: "That confirmation code has expired or does not exist."
+    resend_success_flash:
+      confirmation_sent: We've sent a new confirmation note to %{email} and as soon as you confirm your account you'll be able to get mapping.
+      whitelist: If you use an antispam system which sends confirmation requests then please make sure you whitelist %{sender} as we are unable to reply to any confirmation requests.
   messages:
     inbox:
       title: "Inbox"

diff --git a/test/system/confirmation_resend.rb b/test/system/confirmation_resend.rb
index 69103e67c3567eb830d3947722978ba4174cec9d..6bbde09e0a2dfd86dc5a084245a8d269d30bc498 100644 (file)
--- a/test/system/confirmation_resend.rb
+++ b/test/system/confirmation_resend.rb
@@ -21,6 +21,6 @@ class ConfirmationResendSystemTest < ApplicationSystemTestCase
     visit user_confirm_resend_path(@user)
 
     assert page.has_content?("sent a new confirmation")
-    assert_not page.has_content?("<br />")
+    assert_not page.has_content?("<p>")
   end
 end