[openstreetmap/openstreetmap-website] Add 'canDownloadImage' layer option (PR #5416)

Tom Hughes notifications at github.com
Thu Dec 19 14:54:30 UTC 2024


@tomhughes commented on this pull request.



> @@ -411,34 +416,20 @@ L.OSM.share = function (options) {
       $("#mapnik_image_width").text(mapWidth);
       $("#mapnik_image_height").text(mapHeight);
 
-      const layerId = map.getMapBaseLayerId();
-      const layerKeys = new Map([
-        ["mapnik", "standard"],
-        ["cyclemap", "cycle_map"],
-        ["transportmap", "transport_map"]
-      ]);
+      const canDownloadImage = layer && layer.options.canDownloadImage;

Yes currently we have ECMA 2015 as our version but we should probably discuss what would be an appropriate standard to use these days.

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

Message ID: <openstreetmap/openstreetmap-website/pull/5416/review/2515079330 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20241219/98a6160a/attachment.htm>


More information about the rails-dev mailing list