From 8221005bc61cf1f1e5b7066d257ec80b14eedd35 Mon Sep 17 00:00:00 2001
From: Simon Tharby <simontharby@gmail.com>
Date: Mon, 30 Sep 2024 08:10:37 +0100
Subject: [PATCH] [TAN-2646] Remove alt text from mail logo image

---
 back/app/views/application/_head.mjml                          | 3 +--
 back/app/views/application/_logo_medium_top.mjml               | 3 ++-
 .../app/views/email_campaigns/application/_head.mjml           | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/back/app/views/application/_head.mjml b/back/app/views/application/_head.mjml
index aa98b28131e5..d2b34ef97340 100644
--- a/back/app/views/application/_head.mjml
+++ b/back/app/views/application/_head.mjml
@@ -16,10 +16,9 @@
     <mj-all padding="0" font-family="Arial" font-size="15px" line-height="150%" color="black" />
   </mj-attributes>
 
-  <%#  We add underscores to pad the text displayed in the preview so that the body text (including the logo alt text) is not visible %>
   <% unless preheader.empty? %>
     <mj-preview>
-      <%= preheader + ' _____________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________'%>
+      <%= preheader + ' - '%>
     </mj-preview>
   <% end %>
 </mj-head>
diff --git a/back/app/views/application/_logo_medium_top.mjml b/back/app/views/application/_logo_medium_top.mjml
index 7620d1c2a381..92bdc9884cbf 100644
--- a/back/app/views/application/_logo_medium_top.mjml
+++ b/back/app/views/application/_logo_medium_top.mjml
@@ -2,7 +2,8 @@
   <mj-column>
     <mj-raw>
       <%= link_to home_url do %>
-        <%= image_tag tenant_logo_url, alt: 'Organization logo' %>
+        <%#  We don't add an alt-text, as this will be visible in mail clients mail list, appended to the preheader text %>
+        <%= image_tag tenant_logo_url %>
       <% end %>
     </mj-raw>
   </mj-column>
diff --git a/back/engines/free/email_campaigns/app/views/email_campaigns/application/_head.mjml b/back/engines/free/email_campaigns/app/views/email_campaigns/application/_head.mjml
index 70e56b44bc04..3f7accde9c5f 100644
--- a/back/engines/free/email_campaigns/app/views/email_campaigns/application/_head.mjml
+++ b/back/engines/free/email_campaigns/app/views/email_campaigns/application/_head.mjml
@@ -31,7 +31,7 @@
   <%#  We add underscores to pad the text displayed in the preview so that the body text (including the logo alt text) is not visible %>
   <% unless preheader.empty? %>
     <mj-preview>
-      <%= preheader + ' _________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________________'%>
+      <%= preheader + ' - '%>
     </mj-preview>
   <% end %>
 </mj-head>