@CommanderStorm commented on this pull request.
> @@ -8,11 +8,11 @@
//= require leaflet/dist/leaflet-src
//= require leaflet.osm
//= require leaflet.shortbread
+//= require i18n
We currently have these 3 definitions of which styles exist.
Would it not be nice to have 2 (and only one, once leaflet gets removed):
-
[`layers.yml`](https://github.com/openstreetmap/openstreetmap-website/blob/master/config/layers.yml)
- [`leaflet.osm.js`](vendor/assets/leaflet/leaflet.osm.js)
- the place which I have added
Yes, they don't change all that often, but with 3 places, it is simple to miss
one.
I think given that this is simpler once there are only 2 places, moving this to
the future is likely simpler.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6683#discussion_r2681943578
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6683/review/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
