[openstreetmap/openstreetmap-website] Replace vendored leaflet.polyline plugin with @mapbox/polyline (PR #6996)

[Tom Hughes]

@tomhughes commented on this pull request.



> @@ -7,7 +7,7 @@
 //= require leaflet.legend
 //= require leaflet.note
 //= require leaflet.share
-//= require leaflet.polyline
+//= require polyline_decoder

Well application.js is just included everywhere from the main layout and then a few pages add some extra code that is specific to all them.

Maybe that's not the best way but it's what we have - there's a lot of history here and back when we started a few large bundles was reckoned to be the best approach. These days I think the modern solution is just to load each file individually even in production because HTTP2 make it less of an issue to be loading lots of things.

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

Message ID: <openstreetmap/openstreetmap-website/pull/6996/review/4107876862 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260414/77cac71d/attachment.htm>