[openstreetmap/openstreetmap-website] Use MapLibre class extensions to DRY code (PR #6726)

Marwin Hochfelsner notifications at github.com
Mon Jan 19 17:54:50 UTC 2026


I'd like to seize the opportunity of the API migration to restructure where and how our customisations of it reside.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Extend MapLibre Popup class
  * Extend MapLibre Marker class
  * Move CombinedControlGroup to MapLibre DOM extensions
  * Extend MapLibre NavigationControl class
  * Extend MapLibre GeolocateControl class
  * Extend MapLibre Map class
  * Restructure maplibre requiring hierarchy
  * Change i18n keys to drop maplibre translation patch panel

-- File Changes --

    M app/assets/javascripts/dashboard.js (20)
    M app/assets/javascripts/diary_entry.js (13)
    M app/assets/javascripts/leaflet.locate.js (4)
    M app/assets/javascripts/leaflet.zoom.js (4)
    D app/assets/javascripts/maplibre.i18n.js (7)
    D app/assets/javascripts/maplibre.map.js (93)
    R app/assets/javascripts/maplibre/controls.js (26)
    A app/assets/javascripts/maplibre/dom_util.js (55)
    A app/assets/javascripts/maplibre/map.js (48)
    A app/assets/javascripts/maplibre/styles.js (20)
    M app/assets/javascripts/user.js (18)
    M config/locales/en.yml (12)

-- Patch Links --

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

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

Message ID: <openstreetmap/openstreetmap-website/pull/6726 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260119/2cefe03e/attachment-0001.htm>


More information about the rails-dev mailing list