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

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/3936#discussion_r1114125355">app/views/accounts/_go_public.html.erb</a>:</p>
<pre style='color:#555'>> @@ -0,0 +1,14 @@
+<hr class="mb-3" />
+<a name="public"></a>
+<h2><%= t ".heading" %></h2>
+<p>
+  <%= t ".currently_not_public" %>
+  <strong><%= t ".only_public_can_edit" %></strong>
+  <%= t ".find_out_why_html", :link => link_to(t(".find_out_why"),
+                                               t(".find_out_why_url")) %>
</pre>
<p dir="auto">Lego translations are where we stick two translations together side-by-side, like <code class="notranslate">"string" + "string"</code> which prevents translators from rearranging the sentence to meet any grammatical needs. Interpolating is fine since they can still put the words in whichever order they like, e.g. moving words from after the link text to before the link text.</p>
<div class="highlight highlight-text-html-erb" dir="auto"><pre class="notranslate"><span class="pl-k"><%=</span> t(".first_part") + link_to(foo) + t(".second_part") <span class="pl-k">%></span> # lego translation
vs
<span class="pl-k"><%=</span> t(".whole_sentence"), :link => link_to(foo) <span class="pl-k">%></span><span class="pl-en"></span><span class="pl-kos"></span><span class="pl-s"></span><span class="pl-kos"></span><span class="pl-en"></span><span class="pl-kos"></span><span class="pl-en"></span><span class="pl-kos"></span><span class="pl-en"></span><span class="pl-kos"></span><span class="pl-s"></span><span class="pl-kos"></span><span class="pl-en"></span><span class="pl-kos"></span><span class="pl-s"></span><span class="pl-kos"></span><span class="pl-kos"></span><span class="pl-pds"></span><span class="pl-c1"></span><span class="pl-en"></span><span class="pl-kos"></span><span class="pl-en"></span><span class="pl-kos"></span> # no problem</pre></div>
<p dir="auto">We already use the interpolation approach to links in numerous places.</p>
<p dir="auto">There are two things we should consider with the interpolation approach:</p>
<ul dir="auto">
<li>Don't be tempted to reuse the interpolated link text between different pages, since that text itself might depend on the rest of the sentence</li>
<li>There's a risk that splitting the sentence into two keys makes it harder to translate, e.g. if you are looking at the link text, it could be harder to discover what context it's being used in. So we should keep the names of the keys similar (and keep them in the same naming hierarchy) so that it's more obvious to translators how the keys are related. I try to follow the convention of "blah blah %{foobar_link}", with the <code class="notranslate">foobar</code> key being the link text and <code class="notranslate">foobar_url</code> if the url is translateable.</li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/3936#discussion_r1114125355">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLLX2A6P225VTTL3GDLWYXSH3ANCNFSM6AAAAAAVECLM6U">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLL5I6CQQX6MHIMJEJLWYXSH3A5CNFSM6AAAAAAVECLM6WWGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTSOATR6C.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><openstreetmap/openstreetmap-website/pull/3936/review/1308943329</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/3936#discussion_r1114125355",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/3936#discussion_r1114125355",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>