[openstreetmap/openstreetmap-website] Refactor embed.js.erb with layers.yml (PR #5571)

[mmd]

@mmd-osm commented on this pull request.



>  <% end %>
   };
 
   var map = L.map("map");
   map.attributionControl.setPrefix("");
   map.removeControl(map.attributionControl);
 
-  if (args.layer === "cyclosm") {
-    new L.OSM.CyclOSM().addTo(map);
-  } else if (args.layer === "cyclemap" || args.layer === "cycle map") {
-    new L.OSM.CycleMap(thunderforestOptions).addTo(map);
-  } else if (args.layer === "transportmap") {
-    new L.OSM.TransportMap(thunderforestOptions).addTo(map);
-  } else if (args.layer === "hot") {
-    new L.OSM.HOT().addTo(map);
-  } else {
-    new L.OSM.Mapnik(mapnikOptions).addTo(map);
-  }
+  const layers = <%=
+(YAML.load_file(Rails.root.join("config/layers.yml"))

You could probably use an initializer here to avoid parsing the file all the time: https://github.com/openstreetmap/openstreetmap-website/blob/master/config/initializers/browse_icons.rb

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

Message ID: <openstreetmap/openstreetmap-website/pull/5571/review/2580136819 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250128/1a070d2d/attachment.htm>