Disabled OutputSafety rubocop rule just for the line, not whole browse_helper.rb

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

diff --git a/app/helpers/notifier_helper.rb b/app/helpers/notifier_helper.rb
index 3e53e2543c006a57f9dee08113099abd892df457..dbd2d245d0dbfbccafaaef5a9af7a33188076f0f 100644 (file)
--- a/app/helpers/notifier_helper.rb
+++ b/app/helpers/notifier_helper.rb
@@ -5,10 +5,16 @@ module NotifierHelper
 
   def link_to_user(display_name)
     link_to(
 
   def link_to_user(display_name)
     link_to(

-      display_name,
+      content_tag(
+        "strong",
+        display_name,
+        # NB we need "text-decoration: none" twice: GMail only honours it on
+        # the <a> but Outlook only on the <strong>
+        :style => "text-decoration: none"
+      ),

       user_url(display_name, :host => SERVER_URL),
       :target => "_blank",
       user_url(display_name, :host => SERVER_URL),
       :target => "_blank",

-      :style => "text-decoration: none; color: #222; font-weight: bold"
+      :style => "text-decoration: none; color: #222"

     )
   end
 
     )
   end
 


@@ -20,6 +26,9 @@ module NotifierHelper
   end
 
   def style_message(html)
   end
 
   def style_message(html)

-    html.gsub /<p>/, '<p style="color: black; margin: 0.75em 0">'
+    # 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">'

   end
 end
   end
 end