-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Polygon snapping? #118
Comments
Prototype at https://github.com/dabreegster/route_snapper/tree/cordon. We can feed it any line-strings we want, if there's a desire for non-OSM based |
Ace. I tentatively suggest this is part of the v2 road map. |
Have set it to v2. We can always revisit. |
dabreegster
added a commit
to dabreegster/route_snapper
that referenced
this issue
Apr 27, 2023
dabreegster
added a commit
to dabreegster/route_snapper
that referenced
this issue
Apr 27, 2023
dabreegster
added a commit
to dabreegster/route_snapper
that referenced
this issue
Apr 27, 2023
Merged
This is working, with no urgent next steps known. dabreegster/route_snapper#27 has improvements |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
There are three main options:
option 1: broadly, like the existing route snapping, but as soon as you have 3 points, it connects the first and last. You can no longer click normally to add a totally new point, just drag existing points to make new waypoints
option 2: like route snapper, but some button to say "close it off and make an area". then you can only drag to adjust
option 3: make it more like the current polygon tool, where a click will insert a vertex in the nearest line segment. like that, but try to snap
Which to go for? Comments below welcome (requires signing up for a GitHub account).
Previous outdated comment below.
Worth implementing? I know it would be of use to the spatial planning team although they may be looking for a non OSM source of snap points. It could be handy when planning traffic management areas but not sure. So pitching as a question.The text was updated successfully, but these errors were encountered: