Skip to content
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

Merging an empty with a sided non-empty feature breaks the non-empty feature's side directionality #6033

Closed
RudyTheDev opened this issue Mar 10, 2019 · 0 comments
Assignees
Labels
operation An editing operation / edit menu item
Milestone

Comments

@RudyTheDev
Copy link
Contributor

Scenario: draw a guard rail (case 1 in screenshot), then draw another empty line and connect them going in opposite direction (case 2). Shift-select the guard rail (case 3) and combine the features expecting the new line to become the guard rail (case 4). Observe that the "new" guard rail actually changes the side/direction (case 5).

merge user case side

This is fine when features know how to reverse themselves (and it looks like iD now does this great), but stuff like guard rails are side-dependent and not reversible.

@quincylvania quincylvania added the operation An editing operation / edit menu item label Mar 10, 2019
@bhousel bhousel self-assigned this Mar 22, 2019
@bhousel bhousel added this to the 2.15.0 milestone Mar 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
operation An editing operation / edit menu item
Projects
None yet
Development

No branches or pull requests

3 participants