zlant / parking-lanes

Parking lanes viewer from OpenStreetMap
https://zlant.github.io/parking-lanes/
MIT License
77 stars 13 forks source link

iD Editor Link: Use standalone editor #103

Closed tordans closed 1 year ago

tordans commented 2 years ago

This refactors the iD Editor Link to use the stand alone editor to set more config options via URL params.

The editor function allows to generate an URL based on the map zoom+center or based on a osm object.


My initial goal was to extend this to support custom background layer (so the user sees the same background layer in iD that was selected in this editor before), but this addition requires more changes to the way the background layers are structured; maybe as part of a "allow local layer" refactoring.

zlant commented 2 years ago

Why link to https://ideditor-release.netlify.app instead https://www.openstreetmap.org/edit?editor=id?

tordans commented 1 year ago

Why link to https://ideditor-release.netlify.app/ instead https://www.openstreetmap.org/edit?editor=id?

The stand alone version allows for more customization via URL params and is generally a bit more up to date. But AFAIR right now we don't use the customizations, so we could switch back to the regular version.

Let me know if this is what is holding the PR back.

tordans commented 1 year ago

I rebased this PR on top of the current master just now.

tordans commented 1 year ago

Update: I double checked, the iD Version on osm.org makes it harder to customize the links to include street level images and such. That is why the stand alone version is the better pick.

zlant commented 1 year ago

Rebase yet please

tordans commented 1 year ago

Rebase yet please

I thought I did 🤔 . Will have a look.

tordans commented 1 year ago

@zlant I think this is rebased…

@tordans tordans force-pushed the rework-id-editor-url branch from dff8d79 to 990c8e3 15 minutes ago

zlant commented 1 year ago

I thought I did 🤔 . Will have a look.

I pushed the commits at the same time

tordans commented 1 year ago

Now its rebased again.