[openstreetmap/openstreetmap-website] Remove area layers for changesets completely covering map view (PR #5981)

Anton Khorev notifications at github.com
Tue May 6 01:51:36 UTC 2025


@AntonKhorev commented on this pull request.



>  
-        for (const layer of this._bboxLayers) {
-          layer.updateChangesetLayerBounds(changeset);

Because `updateChangesetsOrder` clears all bbox layers and adds the necessary bboxes again.

Previously it wasn't necessary in response to non-zoom map movement because the bbox layers didn't change. Therefore `updateChangesetLocations` didn't call `updateChangesetsOrder` aka `reorderChangesets`.

Now you can move from a view where an edge of a bbox is visible and its area layer should be added to a view where the same edge is no longer visible and the area layer shouldn't be added.

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

Message ID: <openstreetmap/openstreetmap-website/pull/5981/review/2816635522 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250505/4ccf6199/attachment.htm>


More information about the rails-dev mailing list