#9223 closed enhancement (fixed)
[patch] Allow align in circle for multiple ways
| Reported by: | anonymous | Owned by: | team |
|---|---|---|---|
| Priority: | minor | Milestone: | 14.03 |
| Component: | Core | Version: | tested |
| Keywords: | circle | Cc: |
Description
The current implementation of align in circle can only be used with nodes oder a single closed way. For ease of use with e.g. roundabouts that are split it would be nice if you could select multiple ways that are joined end-to-end and form a closed loop and align them in a circle as if they were one way.
For comparison, the rectify action is already possible on multiple ways.
It is also already possible to use align in circle on an unclosed way.
Attachments (4)
Change History (16)
comment:1 by , 11 years ago
comment:2 by , 10 years ago
I propose a patch allowing align in circle for multiple ways if there can be joined into a polygon.
See bug9223.patch
Patch created from core directory with svn diff.
by , 10 years ago
| Attachment: | bug9223.patch added |
|---|
comment:3 by , 10 years ago
| Summary: | Allow align in circle for multiple ways → [patch] Allow align in circle for multiple ways |
|---|
by , 10 years ago
| Attachment: | bug9223-v2.patch added |
|---|
follow-up: 5 comment:4 by , 10 years ago
I propose a new patch: bug9223-v2.patch with some improvement :
Each node with no other referrers than selected ways is placed regularly.
follow-up: 6 comment:5 by , 10 years ago
Replying to Balaitous:
I propose a new patch: bug9223-v2.patch with some improvement :
Each node with no other referrers than selected ways is placed regularly.
Did you have in mind that this will not work with incomplete data ?
follow-up: 7 comment:6 by , 10 years ago
Replying to skyper:
Replying to Balaitous:
I propose a new patch: bug9223-v2.patch with some improvement :
Each node with no other referrers than selected ways is placed regularly.
Did you have in mind that this will not work with incomplete data ?
It's a good question !
But it's already the case for regular polygon. And you can manually change a node position even if data are incomplete.
I think it is better to regularly align point between exits of a roundabout.
comment:7 by , 10 years ago
Replying to Balaitous:
Replying to skyper:
Replying to Balaitous:
I propose a new patch: bug9223-v2.patch with some improvement :
Each node with no other referrers than selected ways is placed regularly.
Did you have in mind that this will not work with incomplete data ?
It's a good question !
But it's already the case for regular polygon. And you can manually change a node position even if data are incomplete.
I think it is better to regularly align point between exits of a roundabout.
Maybe, at least, a warning about incomplete data would be nice or even deny to perform this action on incomplete data.
by , 10 years ago
| Attachment: | bug9223-v3-simple_notification.patch added |
|---|
display a notification if nodes are outside of download area
by , 10 years ago
| Attachment: | bug9223-v4-dialog.patch added |
|---|
open dialog if nodes are outside of download area
comment:8 by , 10 years ago
I don't think it is very useful. There are no notification when you move a node outside of download area.
But I propose 2 solutions :
- bug9223-v3-simple_notification.patch: A simple warning notification
- bug9223-v4-dialog.patch: A dialog, ask confirmation from user
comment:11 by , 10 years ago
| Milestone: | → 14.03 |
|---|
Meanwhile you can use "select all way nodes" from utilsplugin2 to select all nodes of the ways.