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

<p dir="auto">I've done a new review, including pushing a minor fix for a grammar issue, and just have a couple of comments, both about the preferred provider stuff.</p>
<p dir="auto">The first is with the highlighting of the preferred authentication provider:</p>
<p dir="auto"><a href="https://github.com/openstreetmap/openstreetmap-website/assets/147741/0cbf437b-1869-4b62-80f2-c676854ee00a">image.png (view on web)</a></p>
<p dir="auto">as you can see in that image the text alongside the preferred provider is not well aligned vertically relative to the icon which in turn means the icon is visibly closer to the bottom border than the top one.</p>
<p dir="auto">The second is the positioning of the client specific prompt to action above the main header bar:</p>
<p dir="auto"><a href="https://github.com/openstreetmap/openstreetmap-website/assets/147741/07574042-d024-4e5d-ac93-886316e67ee3">image.png (view on web)</a></p>
<p dir="auto">I don't think it's good to put it above the header like that - nothing else on the site does that or causes the header to move down like that.</p>
<p dir="auto">I would suggest moving it and having it replace the generic "Sign up to get started contributing." text with the client specific call to action. That is obviously a less prominent position (though putting it above the header actually also makes it hard to notice or did for me) so maybe it should be bolded or emphasised in some other way.</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/4455#pullrequestreview-2026655404">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLJZQQCHG6IWNAOESODY7OWXZAVCNFSM6AAAAABBLOL2OWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAMRWGY2TKNBQGQ">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLMO42CVNJK4RKYP4DTY7OWXZA5CNFSM6AAAAABBLOL2OWWGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTTYZRHKY.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/4455/review/2026655404</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/4455#pullrequestreview-2026655404",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/4455#pullrequestreview-2026655404",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>