<p></p>
<p dir="auto">Thanks for the PR, and thanks for your patience too.</p>
<p dir="auto">A couple of things that I thought of while reviewing this PR:</p>
<ul dir="auto">
<li>I don't know if there's any way to avoid specifying the classes for the first and last item in the group? Bootstrap button groups work (somehow, I haven't investigated) just based on the position of the button in the button group</li>
<li>The rounding would be even better if we used the SASS variables from bootstrap - e.g. <code class="notranslate">border-start-start-radius: $border-radius;</code> so that they match the rest of the site automatically (although given the small sizes, it's not currently noticeable</li>
<li>The rounding would be even better than that, if we could use bootstrap classes to control the rounding (so that if e.g. we have responsive rounding radii, it works automatically). But I can't find any description of how to round only one corner with the bootstrap docs, so maybe that's not possible.</li>
</ul>
<p dir="auto">Thanks again!</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/3704#issuecomment-1284143355">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLN4BDV2WMRLD2SOQJLWEADJ3ANCNFSM6AAAAAAQQIS5C4">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLO3PD23EDF3KD73G3TWEADJ3A5CNFSM6AAAAAAQQIS5C6WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSMRJ4PW.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/3704/c1284143355</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/3704#issuecomment-1284143355",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/3704#issuecomment-1284143355",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>