<p></p>
<p><b>@tomhughes</b> requested changes on this pull request.</p>
<hr>
<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5201#discussion_r1974162069">app/assets/stylesheets/common.scss</a>:</p>
<pre style='color:#555'>> @@ -52,6 +52,13 @@ time[title] {
color: $blue;
}
+/* Utility for transparent color */
+
+.text-transparent{
</pre>
<p dir="auto">Space before the brace please.</p>
<hr>
<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5201#discussion_r1974162898">app/views/shared/_language_selector.html.erb</a>:</p>
<pre style='color:#555'>> @@ -0,0 +1,15 @@
+<label class="position-relative d-flex <%= classes %>" role="button">
+ <svg width="20" height="20" fill="currentColor">
+ <path d="M20 18h-1.44a.61.61 0 0 1-.4-.12.81.81 0 0 1-.23-.31L17 15h-5l-1 2.54a.77.77 0 0 1-.22.3.59.59 0 0 1-.4.14H9l4.55-11.47h1.89zm-3.53-4.31L14.89 9.5a11.62 11.62 0 0 1-.39-1.24q-.09.37-.19.69l-.19.56-1.58 4.19zm-6.3-1.58a13.43 13.43 0 0 1-2.91-1.41 11.46 11.46 0 0 0 2.81-5.37H12V4H7.31a4 4 0 0 0-.2-.56C6.87 2.79 6.6 2 6.6 2l-1.47.5s.4.89.6 1.5H0v1.33h2.15A11.23 11.23 0 0 0 5 10.7a17.19 17.19 0 0 1-5 2.1q.56.82.87 1.38a23.28 23.28 0 0 0 5.22-2.51 15.64 15.64 0 0 0 3.56 1.77zM3.63 5.33h4.91a8.11 8.11 0 0 1-2.45 4.45 9.11 9.11 0 0 1-2.46-4.45z" />
+ </svg>
+ <% unless disabled %>
+ <select role="button" class="p-0 position-absolute top-0 start-0 w-100 h-100 language-change-trigger text-transparent bg-transparent <%= classes %>">
</pre>
<p dir="auto">I don't really like the whole thing about overlaying an SVG on a transparent select to be honest but I don't have a good alternative either.</p>
<hr>
<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5201#discussion_r1974169408">config/locales/en.yml</a>:</p>
<pre style='color:#555'>> @@ -1984,6 +1984,8 @@ en:
users:
older: Older Users
newer: Newer Users
+ language_selector:
+ language: English
</pre>
<p dir="auto">I think this is very problematic... We're going to have translators translating this to the localised version of "English" rather than the name of their own language.</p>
<p dir="auto">Also until it is translated languages just don't appear, and we start with a menu that only has english on it and hence looks really silly.</p>
<p dir="auto">Maybe this should have one key for each language (so <code class="notranslate">languages.en</code>, <code class="notranslate">languages.fr</code> etc) with the english version and then people can translate them all even if many of them won't immediately be used and we can either show both versions with one in parentheses in the menu, or we can show the localised version with fallback to english.</p>
<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/5201#pullrequestreview-2648759418">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLNTT4QOPRDVLAJGXTL2R5NUDAVCNFSM6AAAAABODAIFFCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDMNBYG42TSNBRHA">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLOWIAIC2BGGNZLDM3T2R5NUDA5CNFSM6AAAAABODAIFFCWGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTU54DOHU.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/5201/review/2648759418</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/5201#pullrequestreview-2648759418",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/5201#pullrequestreview-2648759418",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>