@hlfan commented on this pull request.


> @@ -100,6 +106,9 @@ OSM = {
       mapParams.lon = params.mlon;
       mapParams.lat = params.mlat;
       mapParams.zoom = params.zoom || 12;
+    } else if (geoURI.pathname) {
+      [mapParams.lat, mapParams.lon] = geoURI.pathname.split(",");

I thought we wouldn't parse these parameters with the move to `new URL` and 
parseFloat takes care of these things anyway.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5736#discussion_r1991795477
You are receiving this because you are subscribed to this thread.

Message ID: 
<openstreetmap/openstreetmap-website/pull/5736/review/2679013...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to