[openstreetmap/openstreetmap-website] Pre-center objects with embed-data (PR #5700)

Marwin Hochfelsner notifications at github.com
Mon Feb 17 23:19:02 UTC 2025


Fixes https://github.com/openstreetmap/iD/issues/10779, closes #5698
Before the regression from #5632, the location source table was:

typ |           |          | hash-loc | !hash-loc
:-: |  :------: |  :-----: | :------: | :------:
n/w |  data-loc |  map-obj | hash-loc | nil
rel | !data-loc |  map-obj | hash-loc | nil
gpx |  data-loc | !map-obj | data-loc | data-loc
nil | !data-loc | !map-obj | map-loc  | map-loc

I changed it to this:

typ |           |          | hash-loc | !hash-loc
:-: |  :------: |  :-----: | :------: | :------:
n/w |  data-loc |  map-obj | hash-loc | data-loc
rel | !data-loc |  map-obj | hash-loc | nil
gpx |  data-loc | !map-obj | data-loc | data-loc
nil | !data-loc | !map-obj | map-loc  | map-loc

And I took the opportunity to restructure the conditions in the spirit of #5632.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Pre-center objects with embed-data

-- File Changes --

    M app/assets/javascripts/edit/id.js.erb (11)

-- Patch Links --

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

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

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


More information about the rails-dev mailing list