[openstreetmap/openstreetmap-website] Changeset highlights in a layer on top of normal bboxes (PR #5940)

Anton Khorev notifications at github.com
Sat Apr 19 16:06:00 UTC 2025


A fix for ["Also, sometimes the hover fails and the bbox ends up not on top of other bboxes"](https://community.openstreetmap.org/t/better-osm-org-a-script-that-adds-useful-little-things-to-osm-org/121670/105), look at the animation of the lower bbox in the second image.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Ignore changeset highlight changes mid-zoom
  * Move changesets layer properties init to init hook
  * Don't update style from setChangesetSidebarRelativePosition()
  * Move bboxes to sublayer of history changesets layer
  * Add _getSidebarRelativeClassName helper method
  * Replace changeset highlight state/style with highlight sublayer

-- File Changes --

    M app/assets/javascripts/index/history-changesets-layer.js (82)
    M app/assets/javascripts/index/history.js (6)

-- Patch Links --

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

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

Message ID: <openstreetmap/openstreetmap-website/pull/5940 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250419/3db06712/attachment-0001.htm>


More information about the rails-dev mailing list