<p></p>
<p><b>@tomhughes</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/2970#discussion_r526692461">app/views/user_mailer/_gpx_description.html.erb</a>:</p>
<pre style='color:#555'>> @@ -1,12 +1,8 @@
-<%= t "user_mailer.gpx_notification.your_gpx_file" %>
-<strong><%= @trace_name %></strong>
-<%= t "user_mailer.gpx_notification.with_description" %>
-<em><%= @trace_description %></em>
-<% if @trace_tags.length>0 %>
-  <%= t "user_mailer.gpx_notification.and_the_tags" %>
-  <em><% @trace_tags.each do |tag| %>
-    <%= tag.tag.rstrip %>
-  <% end %></em>
+<% trace_name = tag.strong(@trace_name) %>
+<% trace_description = tag.em(@trace_description) %>
+<% if @trace_tags.length > 0 %>
+  <% tags = @trace_tags.map { tag.tag.rstrip.join(" ") } %>
</pre>
<p>Did you mean to put that <code>join</code> after the <code>map</code> rather than inside it? Also should it be <code>safe_join</code> or is interpolating it into an <code>_html</code> resource going to have much the same effect></p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/2970#pullrequestreview-534201171">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLKFN3D3LTXW5KNLNDTSQTM6DANCNFSM4T2KJZSA">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AAK2OLNEAIAHJKEZ74BERC3SQTM6DA5CNFSM4T2KJZSKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOD7LUGUY.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/2970#pullrequestreview-534201171",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/2970#pullrequestreview-534201171",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>