<p dir="auto">This updates the new locale menu to use a bootstrap dropdown instead of a simple HTML select so that it's appearance better matches the rest of the site.</p>
<p dir="auto">To do this I had to move the selector out of the secondary nav bar which probably isn't ideal but without that something in bootstrap was grabbing all the list items for the dropdown and moving them to the outer list for the nav bar... That move also causes the icon to move to the right which I think is probably fine but could likely be changed if I knew what I was doing.</p>
<p dir="auto">I'm not sure the effect in small screen mode is ideal either... I did consider moving the language selector to the top to always be visible in that case which I think is fine but possibly a bit weird?</p>
<p dir="auto">Other changes are improving the tests a bit and adding an aria label to the selector button to explain what it is for screen readers (and make it easy to click in tests).</p>
<p dir="auto">New design, large screen, with menu open:</p>
<p dir="auto"><a href="https://github.com/user-attachments/assets/08484d6c-27cd-4a95-b73f-b2bdbb283a62">image.png (view on web)</a></p>
<p dir="auto">and small screen:</p>
<p dir="auto"><a href="https://github.com/user-attachments/assets/dc3d17a9-e827-412d-b333-bbd11417f156">image.png (view on web)</a></p>
<hr>
<h4>You can view, comment on, or merge this pull request online at:</h4>
<p> <a href='https://github.com/openstreetmap/openstreetmap-website/pull/6103'>https://github.com/openstreetmap/openstreetmap-website/pull/6103</a></p>
<h4>Commit Summary</h4>
<ul>
<li><a href="https://github.com/openstreetmap/openstreetmap-website/pull/6103/commits/5e0adcc823299e3dc2058b7265f68747d90c26d6" class="commit-link">5e0adcc</a> Use a bootstrap dropdown for language selection</li>
</ul>
<h4 style="display: inline-block">File Changes </h4> <p style="display: inline-block">(<a href="https://github.com/openstreetmap/openstreetmap-website/pull/6103/files">6 files</a>)</p>
<ul>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/6103/files#diff-f3d75a429346b0f71a92922e7bdc526c74f6211947050df3d331c05d3019e5b8">app/assets/javascripts/language_selector.js</a>
(4)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/6103/files#diff-e86a39e6d0b163f152db8843fc7d647a3f5d648de4a74f1732bef4474b6e5bc8">app/assets/stylesheets/common.scss</a>
(12)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/6103/files#diff-6d2ef58207c92d1533287af551a075122ba98240a861819047a2ae77b7674f0b">app/views/layouts/_header.html.erb</a>
(16)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/6103/files#diff-e8ed8ce386aaf278c52395822b83452d2f56e8e50977a44282a1ebb345342066">app/views/shared/_language_selector.html.erb</a>
(21)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/6103/files#diff-44438ce218f5287c58d0017f965d888715635d94280669896f75841fbd7b4cd7">config/locales/en.yml</a>
(2)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/6103/files#diff-43c6cbf1d8474dfd3889f066ee578a693a1b3ca863f407b07663e0f59102b4ed">test/system/site_test.rb</a>
(28)
</li>
</ul>
<h4>Patch Links:</h4>
<ul>
<li><a href='https://github.com/openstreetmap/openstreetmap-website/pull/6103.patch'>https://github.com/openstreetmap/openstreetmap-website/pull/6103.patch</a></li>
<li><a href='https://github.com/openstreetmap/openstreetmap-website/pull/6103.diff'>https://github.com/openstreetmap/openstreetmap-website/pull/6103.diff</a></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/6103">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLNR7V2RNAVQLDWQYRT3DVEVJAVCNFSM6AAAAAB7LEH3PWVHI2DSMVQWIX3LMV43ASLTON2WKOZTGE2DOMZUGAZDIMY">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLJBJM7STPLBEGAAAFT3DVEVJA5CNFSM6AAAAAB7LEH3PWWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHLXGEZ2M.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/6103</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/6103",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/6103",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>