[openstreetmap/openstreetmap-website] Add coordinate precision function (PR #5581)

Tom Hughes notifications at github.com
Sat Feb 1 15:31:56 UTC 2025


@tomhughes requested changes on this pull request.

Other than the one change I've requested can you squash the two fixup commits down into the first commit please.

> @@ -211,11 +207,16 @@ OSM = {
     return Math.ceil(Math.log10(pixels / degrees));
   },
 
+  cropLocation: function (latLng, zoom = map.getZoom()) {

As the test failures showed the default value here doesn't work (because `map` is not defined) so we should get rid of it.

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

Message ID: <openstreetmap/openstreetmap-website/pull/5581/review/2588353198 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250201/26b5f362/attachment.htm>


More information about the rails-dev mailing list