<p></p>
<p dir="auto">Thanks for the mockup. I didn't realise until reviewing that it's a mockup rather than a proposal (i.e. there's a hardcoded options list, none of the buttons work) so I've converted this to a draft.</p>
<p dir="auto">I think this highlights that we need to change the secondary button colour, since it's quite a dark shade of grey compared to our blue and green colours.</p>
<p dir="auto">I don't think primary is the correct choice for the type-of-route buttons, since that's not actually an action on the form, rather it's a toggle choice. I would like to keep the primary colour for the main action (i.e. "click here to make something happen now").</p>
<ul dir="auto">
<li>Talking of which, there's no button to make the routing happen now. Is that deliberate?</li>
<li>How do you propose to handle situations where there are either more or fewer types of routing available? Different engines might have different lists of options (e.g. off-road cycling, skating, only car etc).</li>
<li>Where do the icons come from? There's no attribution afaict.</li>
<li>The icons seem a bit inconsistent e.g. the weight of the arrows compared to the weight of the cross symbol. Do they all come from the same icon set?</li>
</ul>
<p dir="auto">(For icon sets, perhaps <a href="https://icons.getbootstrap.com/" rel="nofollow">https://icons.getbootstrap.com/</a> would be useful.)</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/3400#issuecomment-1057225749">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLIH7JADIUQ5XHL5OA3U56VE5ANCNFSM5KLDVHAQ">unsubscribe</a>.<br />Triage notifications on the go with GitHub Mobile for <a href="https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675">iOS</a> or <a href="https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub">Android</a>.
<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLLI6LCVTLEIKWDPSRLU56VE5A5CNFSM5KLDVHA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOH4B7YFI.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/3400/c1057225749</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/3400#issuecomment-1057225749",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/3400#issuecomment-1057225749",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>