<p></p>
<p><b>@tomhughes</b> commented on this pull request.</p>
<p dir="auto">I'm not sure I agree with either part of the title of this PR as I don't really see any simplification - quite the opposite if anything. The "always do reverse geocoding" is also misleading because it implies extending an existing use when in fact this introduces reverse geocoding for the first time?</p><hr>
<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5064#discussion_r1747448532">app/assets/javascripts/index/directions-endpoint.js</a>:</p>
<pre style='color:#555'>> input.val(json[0].display_name);
changeCallback();
});
}
+ function getReverseGeocode() {
+ var latlng = endpoint.latlng.clone();
+ var reverseGeocodeUrl = OSM.NOMINATIM_URL + "reverse?lat=" + latlng.lat + "&lon=" + latlng.lng + "&format=json";
+
+ endpoint.geocodeRequest = $.getJSON(reverseGeocodeUrl, function (json) {
+ delete endpoint.geocodeRequest;
+ if (!json || !json.display_name) {
+ input.addClass("is-invalid");
+ alert(I18n.t("javascripts.directions.errors.no_place", { place: endpoint.value }));
</pre>
<p dir="auto">Why should it be an error to route to or from a location that can't be reverse geocoded? Surely we should just stick with the coordinates the user gave in that case? If there is a good reason for it then I definitely think we need to replace <code class="notranslate">alert</code> with something better.</p>
<hr>
<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/5064#discussion_r1747463660">app/assets/javascripts/index/directions-endpoint.js</a>:</p>
<pre style='color:#555'>> input.val(json[0].display_name);
changeCallback();
});
}
+ function getReverseGeocode() {
+ var latlng = endpoint.latlng.clone();
+ var reverseGeocodeUrl = OSM.NOMINATIM_URL + "reverse?lat=" + latlng.lat + "&lon=" + latlng.lng + "&format=json";
</pre>
<p dir="auto">Do we need to control the match precision here? Might this cause a location to "snap" to something a long way away?</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#pullrequestreview-2286764681">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLMZVSM7LUPLZJKXAOLZVHPJPAVCNFSM6AAAAABMKM6E5WVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDEOBWG43DINRYGE">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLP4KMTMDPA52D2ANULZVHPJPA5CNFSM6AAAAABMKM6E5WWGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTUIJVBIS.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/review/2286764681</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#pullrequestreview-2286764681",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/5064#pullrequestreview-2286764681",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>