Fixed alignment & underlining bugs in Outlook / WinMail

diff --git a/app/helpers/notifier_helper.rb b/app/helpers/notifier_helper.rb
index 23ee45648267d826139b8ae3fc690199e5c5696a..da55f6c0e1ef09f86011cab325c6ef09e074f57d 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(
-      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",
-      :style => "text-decoration: none; color: #222; font-weight: bold"
+      :style => "text-decoration: none; color: #222"
     )
   end
 

diff --git a/app/views/layouts/notifier.html.erb b/app/views/layouts/notifier.html.erb
index ee9d2d7166841bd7eacaa2b14134ea59f7a8c173..531575beb2087715b429f39401ae309402d31aa7 100644 (file)
--- a/app/views/layouts/notifier.html.erb
+++ b/app/views/layouts/notifier.html.erb
@@ -13,9 +13,10 @@
                   <%= image_tag attachments["logo.png"].url, alt: "OpenStreetMap", title: "OpenStreetMap", height: "30", width: "30", border: "0" %>
                 </a>
               </td>
-              <td style="width: 550px; padding: 0px; text-align: left">
+              <td style="width: 570px; padding: 0px; text-align: left">
+                <%# NB we need "text-decoration: none" twice: GMail only honours it on the <a> but Outlook only on the <strong> %>
                 <a href="<%= @root_url %>" target="_blank" style="text-decoration: none; color: #000">
-                  <h1 style="font-size: 18px; font-weight: 600; margin: 0; text-align: left">OpenStreetMap</h1>
+                  <strong style="text-decoration: none; font-size: 18px; font-weight: 600; margin: 0; text-align: left">OpenStreetMap</strong>
                 </a>
               </td>
             </tr>