[openstreetmap/openstreetmap-website] don't set map coords if mapParams contains object, but no "map" in hash (PR #5698)

Marwin Hochfelsner notifications at github.com
Mon Feb 17 19:49:33 UTC 2025


hlfan left a comment (openstreetmap/openstreetmap-website#5698)

> in this case, params.set("map", …) must not be called!

Why would you not want to use the coordinates for nodes, ways and notes if these are already loaded in the [site_controller](https://github.com/openstreetmap/openstreetmap-website/blob/483f4c1341ee2b0eb438dc7704e4ba272d1e74fe/app/controllers/site_controller.rb#L86)?
In my experience this ommission adds a noticeable delay to seeing the object, as iD first loads the fully zoomed out view and only later can zoom in.
If this delay can be reduced, I'm happy to remove the server side preset coordinates. 

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

Message ID: <openstreetmap/openstreetmap-website/pull/5698/c2663965884 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250217/39cd1f3c/attachment.htm>


More information about the rails-dev mailing list