Skip to content

Fix creating outline for skinned meshes #106076

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TheoLomeuBraga
Copy link

Closes #106074

now creating an outline mesh for a mesh with skim property will share the skim and skeleton properties for the outline mesh

@TheoLomeuBraga TheoLomeuBraga requested a review from a team as a code owner May 4, 2025 18:45
@akien-mga akien-mga changed the title fix_creating_outline_skined_mesh Fix creating outline for skinned meshes May 5, 2025
@akien-mga akien-mga requested a review from a team May 5, 2025 13:56
@akien-mga akien-mga added this to the 4.5 milestone May 5, 2025
@akien-mga
Copy link
Member

Hello! Thanks for contributing.

I assigned animation maintainers for review as they're most familiar with mesh skinning.

A small form nitpick: The commit message should be amended to be a proper sentence in English (like I did with the PR title). See https://github.com/godotengine/godot/blob/master/CONTRIBUTING.md#format-your-commit-messages-with-readability-in-mind and PR workflow for details on how to format and amend commits.

@TheoLomeuBraga TheoLomeuBraga changed the title Fix creating outline for skinned meshes Fix creating an outline for skinned meshes May 5, 2025
@TheoLomeuBraga TheoLomeuBraga changed the title Fix creating an outline for skinned meshes Fix creating outline for skinned meshes May 5, 2025
@TheoLomeuBraga
Copy link
Author

Hello! Thanks for contributing.

I assigned animation maintainers for review as they're most familiar with mesh skinning.

A small form nitpick: The commit message should be amended to be a proper sentence in English (like I did with the PR title). See https://github.com/godotengine/godot/blob/master/CONTRIBUTING.md#format-your-commit-messages-with-readability-in-mind and PR workflow for details on how to format and amend commits.

sorry i'm new to this

@@ -542,6 +542,13 @@ void MeshInstance3DEditor::_create_outline_mesh() {

MeshInstance3D *mi = memnew(MeshInstance3D);
mi->set_mesh(mesho);

Node *skeleton = node->get_node(node->get_skeleton_path());
Copy link
Member

@fire fire May 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems ok, we have a new MeshInstance3D and we either use the existing path or assume that it's the parent node (as a skeleton3d).

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

precisely

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Work in progress
Development

Successfully merging this pull request may close these issues.

creating outline mesh in a skinned mesh does not automatically associate outline mesh with Skeleton3D
3 participants