Remove require_terms_agreed configuration option

[rails.git] / app / helpers / notifier_helper.rb

diff --git a/app/helpers/notifier_helper.rb b/app/helpers/notifier_helper.rb
index da55f6c0e1ef09f86011cab325c6ef09e074f57d..a696a3d9cd79a38394877192b06226189d3b3888 100644 (file)
--- a/app/helpers/notifier_helper.rb
+++ b/app/helpers/notifier_helper.rb
@@ -6,7 +6,7 @@ module NotifierHelper
   def link_to_user(display_name)
     link_to(
       content_tag(
-        'strong',
+        "strong",
         display_name,
         # NB we need "text-decoration: none" twice: GMail only honours it on
         # the <a> but Outlook only on the <strong>
@@ -14,6 +14,7 @@ module NotifierHelper
       ),
       user_url(display_name, :host => SERVER_URL),
       :target => "_blank",
+      :rel => "noopener",
       :style => "text-decoration: none; color: #222"
     )
   end
@@ -29,6 +30,6 @@ module NotifierHelper
     # Because we can't use stylesheets in HTML emails, we need to inline the
     # styles. Rather than copy-paste the same string of CSS into every message,
     # we apply it once here, after the message has been composed.
-    html.gsub /<p>/, '<p style="color: black; margin: 0.75em 0; font-family: \'Helvetica Neue\', Arial, Sans-Serif">'
+    html.gsub(/<p>/, '<p style="color: black; margin: 0.75em 0; font-family: \'Helvetica Neue\', Arial, Sans-Serif">')
   end
 end