microsoft / MixedRealityToolkit-Unity

This repository is for the legacy Mixed Reality Toolkit (MRTK) v2. For the latest version of the MRTK please visit https://github.com/MixedRealityToolkit/MixedRealityToolkit-Unity
https://aka.ms/mrtkdocs
MIT License
6k stars 2.12k forks source link

objectManipulator's rotation is inverted when using Rotation Axis Constraint on X and Y axis, and a move constraint on all movement. #8947

Closed polygonfuture closed 9 months ago

polygonfuture commented 3 years ago

Describe the bug

If an object has the components listed below, objectManipulator's rotation is inverted when air tapping a point on the object and moving hand left and right.

This behavior is inverse to air tapping and rotating your wrist. The behavior is counter intuitive and feels like its broken.

To reproduce

Steps to reproduce the behavior:

1.Create 3D Game Object -> Cube

  1. Add Object Manipulator
  2. Set the following settings: --One Hand Rotation Mode Near: Rotate About Object Center --Two Hand Rotation Mode Near: Rotate About Object Center
  3. Create Move Axis Constraint with changes: --Constraint On Movement: Everything
  4. Create Rotation Axis Constraint with changes: --Constraint On Rotation: Z and X axis checked
  5. Hit Play
  6. Air tap and hold point on Cube Game Object
  7. Move hand left and right
  8. Rotation is inverted.

If you rotate your wrist however, rotation is as expected.

Expected behavior

Air tapping a point on the cube and moving hand left and right should rotate the cube in the direction your hand moves.

Your setup (please complete the following information)

Target platform (please complete the following information)

Additional context

At the very least rotation should have the option of inverting the behavior.

polygonfuture commented 3 years ago

Any word on if this is going to be worked on / adjusted?

polar-kev commented 3 years ago

@CDiaz-MS, can you check the scope of effort required for this fix?

CDiaz-MS commented 3 years ago

Can confirm the behavior, it does feel unexpected: ObjectManipulatorRotationInversion

polar-kev commented 3 years ago

Let's take a look at this along with #8943 together in the next release since Object Manipulator changes could have effects on other components.

graemeaj commented 2 years ago

Is there any workaround for this issue?

polygonfuture commented 2 years ago

I agree, its been over a year. I'd love to see this resolved. Its very unintuitive UX.

IssueSyncBot commented 9 months ago

We appreciate your feedback and thank you for reporting this issue.

Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.

Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.

Please note that MRTK3 was released in August 2023. It features an all new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visithttps://www.mixedrealitytoolkit.org.

Thank you for your continued support of the Mixed Reality Toolkit!