@hlfan commented on this pull request.
On app/assets/javascripts/leaflet.map.js:
What do you think of swapping the hierarchy of the new files, with
`leaflet.shortbread.js` required in `leaflet.maplibregl.js`?
The current way would need to require `leaflet.maplibregl.js` twice,
once in `leaflet.shortbread.js` and once in a possible
`leaflet.openmaptiles.js`.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6137#discussion_r2209360908
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6137/review/3023305...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
