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

VRM10SpringBoneColliderGroup 改善 #2485

Merged
merged 3 commits into from
Nov 7, 2024

Conversation

ousttrue
Copy link
Contributor

@ousttrue ousttrue commented Nov 7, 2024

OnDrawGizmosSelected で配下の collider を gizmo 描画する。

context menu で子オブジェクトを作って、そっちに移動する。
これは、ひとつの secondary に複数の VRM10SpringBoneColliderGroup がアタッチされていると作業しづらいことに対する対策です。よさそうであれば、後に import 時に secondry の子オブジェクトを自動で作ってもよいかもしれない。

@ousttrue
Copy link
Contributor Author

ousttrue commented Nov 7, 2024

context_menu

return;
}

UnityEditor.Undo.IncrementCurrentGroup();
Copy link
Contributor

Choose a reason for hiding this comment

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

この辺ビルドするとエラーになりません?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

忘れてたー

@ousttrue ousttrue requested a review from Santarh November 7, 2024 15:01
@@ -25,6 +25,7 @@ string GetName()
return Colliders[0].name;
}

#if UNITY_EDITOR
Copy link
Contributor

Choose a reason for hiding this comment

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

本当は ifdef やめて Editor asmdef の方に Editor コードは書きたいところだけど
Gizmo や Menu の周りは結構どうしようもないときはあるんだよなあ。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

editor 側に移動しようかと思ったのだけど、
一個の GameObject に複数の同一 component がささっているときに対象を選択するには、
ContextMenu が楽なのでやむを得ない・・・。

@ousttrue ousttrue merged commit c81242b into vrm-c:master Nov 7, 2024
1 check passed
@ousttrue ousttrue deleted the fix/collidergroup_separate branch November 7, 2024 15:22
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.

2 participants