The name `dom_util.js` doesn't really tell us anything and in fact seems to be 
actively misleading in that it's not really providing any sort of DOM utilities 
as far as I can see.

This splits the file into two, hopefully better named, files, one for each of 
the classes it provides.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Split maplibre/dom_util.js into two better named files

-- File Changes --

    M app/assets/javascripts/maplibre/map.js (3)
    R app/assets/javascripts/maplibre/marker.js (20)
    A app/assets/javascripts/maplibre/popup.js (19)

-- Patch Links --

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

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

Message ID: <openstreetmap/openstreetmap-website/pull/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to