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

Tom Hughes notifications at github.com
Mon May 5 12:43:32 UTC 2025


@tomhughes commented on this pull request.



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

This updating of the bounds got removed at the point where you merged `updateChangesetLocations` into `updateChangesetsGeometry` but it's not obvious why - do we no longer need to update the bounds when updating the geometries?

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

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


More information about the rails-dev mailing list