[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