Show headers in mailer previews only if there are headers present

rails · Feb 8, 2023 · 5eb793a · 5eb793a
1 parent fad9050
commit 5eb793a
Showing 1 changed file with 17 additions and 15 deletions.
diff --git a/railties/lib/rails/templates/rails/mailers/email.html.erb b/railties/lib/rails/templates/rails/mailers/email.html.erb
@@ -125,21 +125,23 @@
         </select>
       </dd>
     <% end %>
-
-    <dt>Headers:</dt>
-    <dd>
-      <details>
-        <summary>Show all headers</summary>
-        <table>
-        <% @email.header_fields.each do |field| %>
-          <tr>
-            <td align="right" style="color: #7f7f7f"><%= field.name %>:</td>
-            <td><%= field.value %></td>
-          </tr>
-        <% end %>
-        </table>
-      </details>
-    </dd>
+
+    <% unless @email.header_fields.nil? || @email.header_fields.empty? %>
+      <dt>Headers:</dt>
+      <dd>
+        <details>
+          <summary>Show all headers</summary>
+          <table>
+          <% @email.header_fields.each do |field| %>
+            <tr>
+              <td align="right" style="color: #7f7f7f"><%= field.name %>:</td>
+              <td><%= field.value %></td>
+            </tr>
+          <% end %>
+          </table>
+        </details>
+      </dd>
+    <% end %>
 
     <dt>EML File:</dt>
     <dd><%= link_to "Download", action: :download %></dd>