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

Hervé Saint-Amand notifications at github.com
Tue Jan 3 23:36:11 UTC 2017


saintamh 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

I suppose the alternative would be not to create a helper, and copy-paste its contents in the templates instead of the call to the helper. Inline the helper function, basically. But if you're happy enough with this, I'm happy too.

-- 
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/b1f775f9/attachment.html>


More information about the rails-dev mailing list