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

Frank Elsinga notifications at github.com
Sun Apr 12 19:49:53 UTC 2026


@CommanderStorm commented on this pull request.



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

> No I believe that it will only be included the first time it is seen.

Are you sure? I think moving the require one level down would incldue JS multiple times..
If this were more than a stupid cpp style `#include`, why would this then happen?

- https://github.com/openstreetmap/openstreetmap-website/pull/6630

Or am I misunderstanding what you mean?

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

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


More information about the rails-dev mailing list