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

Improve orthogonalization action #5999

Merged
merged 9 commits into from
Mar 2, 2019
Merged

Improve orthogonalization action #5999

merged 9 commits into from
Mar 2, 2019

Conversation

bhousel
Copy link
Member

@bhousel bhousel commented Mar 1, 2019

This PR fixes several long-standing issues with the orthogonalization action.

@bhousel
Copy link
Member Author

bhousel commented Mar 1, 2019

So.. actionOrthogonalize can square lines now! which is #5093

square line

However.. We already use the S keyboard shortcut for straightening lines, so it's kind of a problem having 2 menu commands with the same shortcut.

@quincylvania
Copy link
Collaborator

We already use the S keyboard shortcut for straightening lines, so it's kind of a problem having 2 menu commands with the same shortcut.

Could we intelligently choose square instead of straighten for lines when the line "can't be straightened because it bends too much?" I guess there are some instances where both are allowed though, and more importantly the user may mean to do one and get the other. I suggest switching the square shortcut to Q.

@bhousel
Copy link
Member Author

bhousel commented Mar 1, 2019

I suggest switching the square shortcut to Q.

Yeah that would neatly solve the problem, and it's what JOSM uses too .. I'll do it.

@bhousel
Copy link
Member Author

bhousel commented Mar 1, 2019

I guess there are some instances where both are allowed though, and more importantly the user may mean to do one and get the other.

Yeah it's actually really easy to construct a line that can be either squared or straightened... It's nice to have both options now!

square and straighten line

@bhousel
Copy link
Member Author

bhousel commented Mar 1, 2019

I added squaring a single vertex for #2205... It doesn't seem as useful as I thought it would be - just squaring one vertex in isolation of all the other ones - but maybe people will want it.

square a vertex

@bhousel
Copy link
Member Author

bhousel commented Mar 2, 2019

I'm going to merge this.. The test failures seem related to #6001 / #6002 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants