From 852cc9770704a2f8580eac71d81e033fa5444331 Mon Sep 17 00:00:00 2001
From: Herve Saint-Amand <git@saintamh.org>
Date: Thu, 9 Feb 2017 20:38:02 +0000
Subject: [PATCH] Fixed alignment & underlining bugs in Outlook / WinMail

---
 app/helpers/notifier_helper.rb      | 10 ++++++++--
 app/views/layouts/notifier.html.erb |  5 +++--
 2 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/app/helpers/notifier_helper.rb b/app/helpers/notifier_helper.rb
index 23ee45648..da55f6c0e 100644
--- 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 ee9d2d716..531575beb 100644
--- 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>
-- 
2.43.2