[VRMC_springBone_limit] fix editor ui

[ousttrue]

fixed #2831

Add joints でLimit設定をコピーするか、コピーしないことをUI上で明示する

は、良くわからなかったので特に作業していません。

[Copilot]

Pull request overview

Updates the VRM 1.0 SpringBone AngleLimit inspector UX to better reflect the current draft spec and reduce user confusion around limit settings (per issue #2831).

Changes:

• Update the AngleLimit foldout label and remove outdated “import/export not implemented” messaging.
• Clarify Cone/Hinge limit UI by labeling m_pitch as “Angle”, and add a warning for end-of-chain (last tail) joints.
• Add constructor parameter documentation notes for AngleLimit fields in BlittableJointMutable.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
Packages/VRM10/Editor/Components/SpringBone/VRM10SpringBoneJointEditor.cs	Adjusts AngleLimit inspector labeling/warnings and removes outdated HelpBox text.
Packages/UniGLTF/Runtime/SpringBoneJobs/Blittables/BlittableJointMutable.cs	Adds documentation notes clarifying AngleLimit parameter meaning.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[0b5vr]

MAY: "末端ノードではAngleLimitは無視されます。" と明確に書いてあげたほうが親切かも。また、設定をコピーして回るときに毎回Noneに変えるのも面倒なので、WarningでなくInfoで良いかも。

[0b5vr]

MUST: isLastTail が true のときにTypeを変更するUIが表示されなくなるのはCopilotの指摘通り。これは修正するべきです。

[0b5vr]

#2834 (comment) と同一の指摘です。

[0b5vr]

Add joints でLimit設定をコピーするか、コピーしないことをUI上で明示する

は、良くわからなかったので特に作業していません。

Add jointsについて、これは VRM10SpringBoneJoint のコンテキストメニューより実行できる "Add joints" について、これがSpringBoneのstiffness, dragForce等の設定はコピーするが、limitの情報はコピーしないという指摘です。言葉足らずですいません 🙇

UniVRM/Packages/VRM10/Runtime/Components/SpringBone/VRM10SpringBoneJoint.cs

Lines 62 to 134 in 52e1250

	void AddJointRecursive(Transform t, VRM10SpringBoneJoint src)
	{
	var joint = t.gameObject.GetOrAddComponent<VRM10SpringBoneJoint>();
	// copy settings
	joint.m_stiffnessForce = src.m_stiffnessForce;
	joint.m_gravityPower = src.m_gravityPower;
	joint.m_gravityDir = src.m_gravityDir;
	joint.m_dragForce = src.m_dragForce;
	joint.m_jointRadius = src.m_jointRadius;
	if (t.childCount > 0)
	{
	// only first child
	AddJointRecursive(t.GetChild(0), src);
	}
	}
	void GetJoints(Transform t, List<VRM10SpringBoneJoint> joints)
	{
	if (t.TryGetComponent<VRM10SpringBoneJoint>(out var joint))
	{
	joints.Add(joint);
	}
	if (t.childCount > 0)
	{
	GetJoints(t.GetChild(0), joints);
	}
	}
	[ContextMenu("Add joints")]
	private void AddJointsToChild0()
	{
	var root = GetComponentInParent<Vrm10Instance>();
	if (root == null)
	{
	UniGLTFLogger.Warning("not Vrm10Instance");
	return;
	}
	if (transform.childCount == 0)
	{
	UniGLTFLogger.Warning("no children");
	return;
	}
	AddJointRecursive(transform.GetChild(0), this);
	// updater root
	foreach (var spring in root.SpringBone.Springs)
	{
	for (int i = 0; i < spring.Joints.Count; ++i)
	{
	if (spring.Joints[i] == this)
	{
	// found
	while (spring.Joints.Count - 1 > i)
	{
	// remove after this joint
	spring.Joints.RemoveAt(spring.Joints.Count - 1);
	}
	// get descendants joints
	var joints = new List<VRM10SpringBoneJoint>();
	GetJoints(transform.GetChild(0), joints);
	// add jonits to after this
	spring.Joints.AddRange(joints);
	return;
	}
	}
	}
	UniGLTFLogger.Warning($"{this} is found in {root}");

[0b5vr]

兼ねCopilotの指摘に同意します。詳細は各コメントを参照ください。