[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