SinghSwayam left a comment (openstreetmap/openstreetmap-website#6669)
@mmd-osm @HolgerJeromin @hlfan Thank you for the guidance! I have refactored
the implementation as requested.
**Changes in this update:**
1. Moved all Wiki URLs into `config/layers.yml`.
2. Added the translation key `layer_info` to `config/locales/en.yml`.
3. Implemented the icon rendering directly in
`app/views/layers_panes/show.html.erb`.
4. Completely removed the JavaScript logic from `leaflet.layers.js`.
It is ready for a re-review when you have a moment. Thanks!
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6669#issuecomment-3706855783
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6669/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
