[openstreetmap/openstreetmap-website] Remove area layers for changesets completely covering map view (PR #5981)
Anton Khorev
notifications at github.com
Tue May 6 02:50:42 UTC 2025
AntonKhorev left a comment (openstreetmap/openstreetmap-website#5981)
> Is there a specific reason you do this in pixel coordinates instead of lat/lon coordinates where you can use `changeset.bounds.contains(map.getBounds())`?
In pixels I take bbox border widths into account, which is maybe not a big difference.
I can use `.contains` (and `.overlaps`) in pixel coordinates too, which I'm going to do for now. Those manual comparisons that I'm removing may come back if I'll do another feature to indicate where bboxes are located relative to the map view.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5981#issuecomment-2853129199
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/5981/c2853129199 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250505/ad73fd75/attachment.htm>
More information about the rails-dev
mailing list