<p></p>
<p dir="auto">Thanks for great PR! The sources are now much more readable and it seems a lot of bugs are fixed.</p>
<p dir="auto">Just a few remarks:</p>
<ol dir="auto">
<li>
<p dir="auto">Perhaps it would be good to fit map's bounds (directions.js Ln 159) on union of markers and route polyline (instead of route polyline only). 2 cases when this is useful:<br>
a) at least one (or both) of markers are too far from retrieved route and they are not visible in final display and<br>
b) routing engine was unable to return the route (e.g. place markers at (0,0) and (1,1)) and only markers are displayed (but, if you type them, map didn't focus on them).</p>
</li>
<li>
<p dir="auto">It seems we still have duplicated calls to nominatim when we select from context menu (other cases are OK). Please, take a look at below video:</p>
</li>
</ol>
<p dir="auto"><a href="https://github.com/user-attachments/assets/2f7a4ed8-1e82-4eb4-becf-fd28a7c3e05c">https://github.com/user-attachments/assets/2f7a4ed8-1e82-4eb4-becf-fd28a7c3e05c</a></p>
<p dir="auto">Anyway, even with above remarks, I think this is great PR. Thanks for making it!</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/5064#issuecomment-2285981589">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLPZFIJUJ3P3WRUZFELZRHSMJAVCNFSM6AAAAABMKM6E5WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBVHE4DCNJYHE">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLOAVLBFT7GYIFLNVWDZRHSMJA5CNFSM6AAAAABMKM6E5WWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUIIFHZK.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/5064/c2285981589</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/5064#issuecomment-2285981589",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/5064#issuecomment-2285981589",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>