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/5...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to