-
Notifications
You must be signed in to change notification settings - Fork 101
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
grid and scene snapping #313
Conversation
crap. i committed a file which shouldn't have been. will update |
Interesting how it allowed the first commit through. Does it choose a couple of things at random to complain about? :) |
I'll see if I can resolve the complexity issue tonight. Might be a good thing so the snapping can be applied to other transforms in the future. |
Finally. Didn't want to autoformat, because that brought so many more "errors". Better to refactor that separately. Feel free to squash commit this. |
Refactored a bit now that I have a better understanding of the architecture |
There are also two types of scene snapping (that I can think of). I guess the one I implemented is the more commonly requested, "snap to ground". The other would be "mouse pick snap" where you snap to whatever you click on. Most of that is already implemented in the PickManager class, so it could easily be added as well. |
Realized there's some work left; I'll change so that snapping only applies to the axis/axes you're editing. I'll also add 15 degree rotation snapping if grid snapping is enabled (unless there are complaints). Not 100% intuitive, but a useful feature, I think. |
Issues
Heh, Codacy <3 Thoughts and prayers... |
(╯°□°)╯︵ ┻━┻ |
It would be so good to have static code analysis that can be also used in the IDE, with the same ruleset... |
Very nice, thank you for your contribution! |
A feature which for some reason was only half implemented(?). I know I have wished for this many times.