-
-
Notifications
You must be signed in to change notification settings - Fork 73
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
3D View - Sculpt Mode - Sculpt header legacy - new Line Trim #4252
Comments
Draise14
added a commit
to Bforartists/files
that referenced
this issue
Jun 5, 2024
Draise14
added a commit
to Bforartists/Manual
that referenced
this issue
Jun 5, 2024
Draise14
added a commit
that referenced
this issue
Jun 5, 2024
…ulpt-header-legacy-new-line-trim 3D View - Sculpt Mode - Sculpt header legacy - new Line Trim #4252
Draise14
added a commit
that referenced
this issue
Jun 5, 2024
#4252 Uploaded the icon dat files. Got missed with the last commit.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
New line trim act-once operator... that already exists in the toolshelf.
Add icon, document.
Details
SHA-1: d4a6164
This PR adds the Line Trim tool to Sculpt mode. It is exposed via
the toolbar along with the other Trim tools and as an entry in the
Sculpt menu.
Technical Approach
Line gestures are represented as two points in screen space: the
start
and
end
of the gesture. Trim tools work by taking a set of screenpoints as the top face of a volume, projecting a copy of the face into
the scene, then using that as the operand shape for boolean operations.
To behave as users would expect, the Line Trim tool does the following
steps to make the initial face:
initial line is long enough to fully divide the object.
and end points.
Differences with Other Trim Tools
corresponding tool options have been disabled in the header.
Alternatives
a bisect operation when using the Fixed projection mode. While this
may result in a better performing tool, it is not guaranteed and
requires extra work to integrate this approach.
Pull Request: https://projects.blender.org/blender/blender/pulls/120845
The text was updated successfully, but these errors were encountered: