[openstreetmap/openstreetmap-website] Rework map element click handling (PR #6281)

Marwin Hochfelsner notifications at github.com
Sat Aug 2 02:00:34 UTC 2025


Yet another PR for click handling.
This fixes #6264 and probably solves #6276 too, but due to the nature of that issue, I haven't been able to test that.
In contrast to #6267, this approach does not make assumptions about what modifiers do.

It also doesn't make assumptions about the event type. Perhaps it should.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Replace links on map with new OSM.router.click() method
  * Make map overlay clicks respect modifier keys

-- File Changes --

    M app/assets/javascripts/index/history-changesets-layer.js (8)
    M app/assets/javascripts/index/layers/data.js (7)
    M app/assets/javascripts/index/layers/notes.js (2)
    M app/assets/javascripts/index/search.js (8)
    M app/assets/javascripts/router.js (11)

-- Patch Links --

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

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

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


More information about the rails-dev mailing list