[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