From: Herve Saint-Amand <git@saintamh.org>
Date: Thu, 9 Feb 2017 21:39:49 +0000 (+0000)
Subject: Make the mails squeezable on narrow screens
X-Git-Tag: live~3542^2~6
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/82a6bb0d5afb3013a1a91a84e129ebd5a0e7dc5f

Make the mails squeezable on narrow screens
---

diff --git a/app/views/layouts/notifier.html.erb b/app/views/layouts/notifier.html.erb
index 531575beb..1ab9493ba 100644
--- a/app/views/layouts/notifier.html.erb
+++ b/app/views/layouts/notifier.html.erb
@@ -6,14 +6,15 @@
     <table style="background-color: #eee; width: 100%">
       <tr>
         <td style="text-align: center; padding: 0px 7px">
-          <table style="width: 600px; color: #222; margin-left: auto; margin-right: auto">
+          <table style="max-width: 600px; color: #222; margin-left: auto; margin-right: auto">
             <tr>
               <td style="width: 30px; padding: 10px 10px 10px 0px">
                 <a href="<%= @root_url %>" target="_blank">
                   <%= image_tag attachments["logo.png"].url, alt: "OpenStreetMap", title: "OpenStreetMap", height: "30", width: "30", border: "0" %>
                 </a>
               </td>
-              <td style="width: 570px; padding: 0px; text-align: left">
+              <%# the "width: 100%" here looks wrong, but I couldn't find a better way of making Outlook give this cell full width %>
+              <td style="width: 100%; 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">
                   <strong style="text-decoration: none; font-size: 18px; font-weight: 600; margin: 0; text-align: left">OpenStreetMap</strong>
diff --git a/app/views/notifier/_message_body.html.erb b/app/views/notifier/_message_body.html.erb
index d66b31aae..485cd3c44 100644
--- a/app/views/notifier/_message_body.html.erb
+++ b/app/views/notifier/_message_body.html.erb
@@ -1,4 +1,4 @@
-<table style="font-size: 15px; margin: 15px 0px; background-color: #eee; width: 565px">
+<table style="font-size: 15px; margin: 15px 0px; background-color: #eee; max-width: 565px">
   <tr>
     <td style="width: 50px; vertical-align: top; padding: 15px">
       <%= link_to(
@@ -14,7 +14,7 @@
           :target => "_blank"
       ) %>
     </td>
-    <td style="text-align: left; vertical-align: top; padding-right: 10px">
+    <td style="text-align: left; vertical-align: top; padding-right: 10px; max-width: 550px">
       <%= body %>
     </td>
   </tr>