-
Notifications
You must be signed in to change notification settings - Fork 93
feat: Solve following mouse pointer #8991
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
Draft
jtran
wants to merge
8
commits into
sketch-revamp
Choose a base branch
from
jtran/solve-following-pointer
base: sketch-revamp
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
275a82c to
4752813
Compare
4752813 to
b9ff5bb
Compare
76f7399 to
178b835
Compare
178b835 to
44fae63
Compare
44fae63 to
62db938
Compare
62db938 to
7155c3f
Compare
7155c3f to
2650266
Compare
2650266 to
ee99a49
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When the user edits segments, assume this is the mouse or other pointing device dragging segments. When executing points, either explicit ones via
point()or implicit ones vialine(), if they are in the dragged set, create an optional constraint that is that point's coordinate fixed to the initial value of the x and yvars. Then, when solving, use ezpz's newsolve_with_priority()KittyCAD/ezpz#147 with required constraints at priority 0 and optional constraints at level 1.