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

make sure DrawLine mode is called with a clean pre-operation graph #2936

Merged
merged 1 commit into from Jan 26, 2016

Conversation

tyrasd
Copy link
Member

@tyrasd tyrasd commented Jan 26, 2016

Potential fix for #2303.

@tyrasd
Copy link
Member Author

tyrasd commented Jan 26, 2016

@jfirebaugh can you please take a look and review this PR (considering the changes in 8a215ca#diff-dca22f16b0f009e9ff9c946ca989cb99)?

@jfirebaugh
Copy link
Member

Yes, I see what's going on -- drawWay.cancel restores this baseGraph, so it needs to be the graph state before adding the pending way.

The reason I could never reproduce this issue is that Mac keyboards do not have a key that triggers this action. They have a "delete" key but it does what a PC keyboard calls "backspace".

tyrasd added a commit that referenced this pull request Jan 26, 2016
make sure DrawLine mode is called with a clean pre-operation graph
@tyrasd tyrasd merged commit 7c9bc50 into openstreetmap:master Jan 26, 2016
@tyrasd tyrasd deleted the patch-2303 branch January 26, 2016 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants