[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