Skip to content
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

Fix Skeleton3D & PhysicalBone3D editor errors #76592

Merged
merged 1 commit into from
May 11, 2023

Conversation

spanzeri
Copy link
Contributor

I was looking at a separate issue, but I run into 2 errors with 3d skeleton and physical bones.
3d skeleton was the "attempting to load..." theme error because the editor was created in construction.
Physical bone 3d was never updated and did not correctly handle calls from EditorNode::_plugin_over_edit

@spanzeri spanzeri requested a review from a team as a code owner April 29, 2023 21:22
@Calinou Calinou added this to the 4.1 milestone Apr 29, 2023
@fire fire requested a review from a team April 29, 2023 22:36
Copy link
Member

@adamscott adamscott left a comment

Choose a reason for hiding this comment

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

Looks good to me with my suggested change. I wouldn't merge without the approval of a member of the animation team though.

Copy link
Member

@akien-mga akien-mga left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@akien-mga akien-mga merged commit 3fdf555 into godotengine:master May 11, 2023
@akien-mga
Copy link
Member

Thanks!

@akien-mga
Copy link
Member

Cherry-picked for 4.0.3.

@akien-mga akien-mga changed the title Fix skeleton_3d & physical_bone_3d editor errors Fix Skeleton3D & PhysicalBone3D editor errors May 12, 2023
akien-mga added a commit that referenced this pull request May 15, 2023
lyuma pushed a commit to V-Sekai/godot that referenced this pull request May 18, 2023
…eleton-editor

Fix Skeleton3D editor crash regression after godotengine#76592
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants