-
Notifications
You must be signed in to change notification settings - Fork 2
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
Can we snap to (active and inactive) objects? #22
Comments
No you're correct, there's no snapping anywhere, and I'm not planning on implementing it, it's a very large feature. The idiomatic way to join two scraps is with the |
My idea of best practice is to snap all conceptually coincident objects/points together, especially scrap/passage ends/joins, but also for any other point or line. Avoids potential for hard to find issues that can arise when Therion tries to identify the interior of passages and areas. Avoids gaps opening up due to morphing as well. |
Hi again! I think I've implemented the minimum usable version of the functionality you were looking for, as now the Detail select tool can snap segments to other lines. This will only work if you're
I'm pretty sure this is enough for joining lines together, and in the future I may add this to the Select tool where it will work on points. But I'm still not sure how it would be possible to snap multiple segments or objects when moving them. |
Often it is desirable to snap objects being drawn to existing objects in an inactive scrap.
The walls at the junction between two scraps is a common example. This avoids having to manually invoke a scrap join elsewhere in ones Therion input data.
I might have missed something, but I cannot get the start or ends of lines, or new points to snap to control points (such as the ends of lines, line points or other points) on other objects.
The only snapping I can get at the moment is snapping to close the line that is currently being drawn, and snapping (tolerance) with the Inspect tool. Am I having early morning brain fade perhaps? :)
The text was updated successfully, but these errors were encountered: