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

Perfect squares cannot be squared as they are not squarish #5967

Closed
rivermont opened this issue Feb 24, 2019 · 3 comments · Fixed by #5999
Closed

Perfect squares cannot be squared as they are not squarish #5967

rivermont opened this issue Feb 24, 2019 · 3 comments · Fixed by #5999
Assignees
Labels
operation An editing operation / edit menu item usability An issue with ease-of-use or design
Milestone

Comments

@rivermont
Copy link
Contributor

Currently when a perfectly square way (such as this one) is attempted to be squared by either s or right click + square, it returns the message This can't be made square because it is not squarish.

This can be confusing as it obviously looks square and the user may experience an existential crisis as a result of being told a square is not squarish. This would seem to be a race case but it would be nice to have a condition of some sort that gives a different message or just squares it anyway.

@quincylvania quincylvania added usability An issue with ease-of-use or design operation An editing operation / edit menu item labels Feb 25, 2019
@bhousel bhousel self-assigned this Feb 26, 2019
@bhousel
Copy link
Member

bhousel commented Feb 26, 2019

Ah yeah, this is because the code uses dot products to check for squareness, but a perfect 90° angle will have a dot product of 0.

I'm going to take a pass over the orthogonalizing code today and try to fix this and a few old related issues.. (#1979, #2472)

@quincylvania
Copy link
Collaborator

We should also be smart with feedback on straightening lines if they're already straight.

@bhousel
Copy link
Member

bhousel commented Mar 1, 2019

We should also be smart with feedback on straightening lines if they're already straight.

Yeah - I almost put this into #5999 but I'm holding off until I merge #5740 to avoid conflicting @jguthrie100's work

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 usability An issue with ease-of-use or design
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants