[openstreetmap/openstreetmap-website] Rename 'keyid' layer option to 'layerId' (PR #5389)

Anton Khorev notifications at github.com
Mon Dec 9 22:31:09 UTC 2024


Each base map layer has a `keyid`. It was introduced in https://github.com/openstreetmap/openstreetmap-website/commit/79df86023c5db1f1c47c80808565733c1200656a to hide/show map key entries for different layers. But since then it was also used for various other things and the function to get it is `getMapBaseLayerId`. So it's a `layerId` actually.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/5389

-- Commit Summary --

  * Rename 'keyid' layer option to 'layerId'

-- File Changes --

    M app/assets/javascripts/index.js (2)
    M app/assets/javascripts/leaflet.map.js (4)
    M config/layers.yml (12)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/5389.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5389.diff

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

Message ID: <openstreetmap/openstreetmap-website/pull/5389 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20241209/f40a4682/attachment.htm>


More information about the rails-dev mailing list