[openstreetmap/openstreetmap-website] Dressed up notification mails (#1401)

Tom Hughes notifications at github.com
Tue Jan 3 23:01:54 UTC 2017


tomhughes commented on this pull request.



> @@ -2,4 +2,13 @@ module NotifierHelper
   def fp(text)
     format_paragraph(text, 72, 0)
   end
+
+  def link_to_user(display_name)
+    link_to(
+      display_name,
+      user_url(display_name, :host => SERVER_URL),
+      :target => "_blank",
+      :style => "text-decoration: none; color: #222; font-weight: bold"
+    )
+  end
 end

Is there something wrong with what you've done? I mean we would normally avoid inline CSS completely but for email that's not possible so I don't see what other option there is?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/1401
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20170103/3f825606/attachment-0001.html>


More information about the rails-dev mailing list