@AntonKhorev commented on this pull request.


> @@ -369,6 +368,14 @@ L.extend(L.Icon.Default.prototype, {
   }
 });
 
+OSM.boundsArea = function ([[minLat, minLon], [maxLat, maxLon]]) {
+  const height = maxLat - minLat;
+  let width = maxLon - minLon;
+
+  if (width < 0) width += 360;

This fix is specific to what Overpass returns. You want to put it into a 
generic area calculation. Do you think it's going to work for all possible 
bounding boxes?

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

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

Reply via email to