@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/2814675...@github.com>_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
