Fix translation handle rotation

MaxWang-MS · MaxWang-MS · commit 312bcdc2c5a0 · 2021-02-19T18:52:25.000-08:00
diff --git a/Assets/MRTK/SDK/Features/UX/Scripts/BoundsControl/Visuals/TranslationHandles.cs b/Assets/MRTK/SDK/Features/UX/Scripts/BoundsControl/Visuals/TranslationHandles.cs
@@ -46,11 +46,11 @@ protected override Quaternion GetRotationRealignment(int handleIndex)
             // Align handle with its edge assuming that the prefab is initially aligned with the up direction 
             if (VisualUtils.FaceAxisType[handleIndex] == CardinalAxisType.X)
             {
-                return Quaternion.FromToRotation(Vector3.up, directionSign * Vector3.right);
+                return Quaternion.FromToRotation(Vector3.forward, directionSign * Vector3.right);
             }
             else if (VisualUtils.FaceAxisType[handleIndex] == CardinalAxisType.Z)
             {
-                return Quaternion.FromToRotation(Vector3.up, directionSign * Vector3.forward);
+                return Quaternion.FromToRotation(Vector3.forward, directionSign * Vector3.forward);
             }
             else // y axis
             {

Original file line number	Diff line number	Diff line change
`@@ -46,11 +46,11 @@ protected override Quaternion GetRotationRealignment(int handleIndex)`
`46`	`46`	`// Align handle with its edge assuming that the prefab is initially aligned with the up direction`
`47`	`47`	`if (VisualUtils.FaceAxisType[handleIndex] == CardinalAxisType.X)`
`48`	`48`	`{`
`49`		`- return Quaternion.FromToRotation(Vector3.up, directionSign * Vector3.right);`
	`49`	`+ return Quaternion.FromToRotation(Vector3.forward, directionSign * Vector3.right);`
`50`	`50`	`}`
`51`	`51`	`else if (VisualUtils.FaceAxisType[handleIndex] == CardinalAxisType.Z)`
`52`	`52`	`{`
`53`		`- return Quaternion.FromToRotation(Vector3.up, directionSign * Vector3.forward);`
	`53`	`+ return Quaternion.FromToRotation(Vector3.forward, directionSign * Vector3.forward);`
`54`	`54`	`}`
`55`	`55`	`else // y axis`
`56`	`56`	`{`