Update pointer visualizer to set default cursor distance

Fixes #2166.

Author: keveleigh

Assets/HoloToolkit/Input/Prefabs/LinearControllerPointer.prefab

@@ -62,7 +62,6 @@ MonoBehaviour:
   ScaleOffset: {x: 1, y: 1, z: 1}
   SetScaleOnAttach: 0
   CurrentPointerOrientation: 0
-  extentOverride: 2
   RaycastOrigin: {fileID: 0}
   LineColorSelected:
     serializedVersion: 2

Assets/HoloToolkit/Input/Scripts/Cursor/Cursor.cs

@@ -20,7 +20,19 @@ public abstract class Cursor : MonoBehaviour, ICursor
         /// <summary>
         /// The pointer that this cursor should follow and process input from.
         /// </summary>
-        public IPointingSource Pointer { get; set; }
+        public IPointingSource Pointer
+        {
+            get { return pointer; }
+            set
+            {
+                pointerIsInputSourcePointer = value is InputSourcePointer;
+                pointer = value;
+            }
+        }
+
+        private IPointingSource pointer;
+
+        private bool pointerIsInputSourcePointer = false;
 
         /// <summary>
         /// Minimum distance for cursor if nothing is hit
@@ -110,6 +122,8 @@ public Vector3 LocalScale
         private Vector3 targetScale;
         private Quaternion targetRotation;
 
+        private float originalDefaultCursorDistance;
+
         /// <summary>
         /// Indicates if the cursor should be visible
         /// </summary>
@@ -126,6 +140,8 @@ public bool IsVisible
 
         private void Awake()
         {
+            originalDefaultCursorDistance = DefaultCursorDistance;
+
             // Use the setter to update visibility of the cursor at startup based on user preferences
             IsVisible = isVisible;
             SetVisibility(isVisible);
@@ -298,6 +314,21 @@ protected virtual void UpdateCursorTransform()
                 TargetedObject = null;
                 TargetedCursorModifier = null;
 
+                if (pointerIsInputSourcePointer)
+                {
+                    // This value get re-queried every update, in case the app has
+                    // changed the pointing extent of the pointer for the current scenario.
+                    float distance = FocusManager.Instance.GetPointingExtent(Pointer);
+                    if (DefaultCursorDistance != distance)
+                    {
+                        DefaultCursorDistance = distance;
+                    }
+                }
+                else if (DefaultCursorDistance != originalDefaultCursorDistance)
+                {
+                    DefaultCursorDistance = originalDefaultCursorDistance;
+                }
+
                 targetPosition = RayStep.GetPointByDistance(Pointer.Rays, DefaultCursorDistance);
                 lookForward = -RayStep.GetDirectionByDistance(Pointer.Rays, DefaultCursorDistance);
                 targetRotation = lookForward.magnitude > 0 ? Quaternion.LookRotation(lookForward, Vector3.up) : transform.rotation;

Assets/HoloToolkit/Input/Scripts/Focus/InputSourcePointer.cs

@@ -129,7 +129,7 @@ public virtual void OnPostRaycast()
         {
             if (PointerRay != null)
             {
-                PointerRay.UpdateRenderedLine(rays, Result, selectPressed);
+                PointerRay.UpdateRenderedLine(rays, Result, selectPressed, FocusManager.Instance.GetPointingExtent(this));
             }
         }
 

Assets/HoloToolkit/Input/Scripts/Focus/SimpleSinglePointerSelector.cs

@@ -30,14 +30,14 @@ public class SimpleSinglePointerSelector : MonoBehaviour, ISourceStateHandler, I
         private bool searchForCursorIfUnset = true;
         public bool SearchForCursorIfUnset { get { return searchForCursorIfUnset; } set { searchForCursorIfUnset = value; } }
 
-        [Tooltip("If true, always select the best pointer available (OS behaviour does not autoselect).")]
+        [Tooltip("If true, always select the best pointer available (OS behavior does not auto-select).")]
         [SerializeField]
         private bool autoselectBestAvailable = false;
         public bool AutoselectBestAvailable { get { return autoselectBestAvailable; } set { autoselectBestAvailable = value; } }
 
         [Tooltip("The line pointer prefab to use, if any.")]
         [SerializeField]
-        private GameObject linePointerPrefab;
+        private GameObject linePointerPrefab = null;
 
         private PointerLine instantiatedPointerLine;
 

Assets/HoloToolkit/Input/Scripts/Utilities/BaseControllerPointer.cs

@@ -10,20 +10,10 @@ public abstract class BaseControllerPointer : AttachToController
     [Range(0f, 360f)]
     protected float CurrentPointerOrientation;
 
-    [SerializeField]
-    [Range(0.5f, 50f)]
-    private float extentOverride = 2f;
-
     [SerializeField]
     [Tooltip("Source transform for raycast origin - leave null to use default transform")]
     protected Transform RaycastOrigin;
 
-    public float? ExtentOverride
-    {
-        get { return extentOverride; }
-        set { extentOverride = value ?? FocusManager.Instance.GlobalPointingExtent; }
-    }
-
     /// <summary>
     /// The Y orientation of the pointer target - used for touchpad rotation and navigation
     /// </summary>

Assets/HoloToolkit/Input/Scripts/Utilities/PointerLine.cs

@@ -67,7 +67,7 @@ protected override void OnEnable()
             LineBase.enabled = false;
         }
 
-        public void UpdateRenderedLine(RayStep[] lines, PointerResult result, bool selectPressed)
+        public void UpdateRenderedLine(RayStep[] lines, PointerResult result, bool selectPressed, float extent)
         {
             if (LineBase == null) { return; }
 
@@ -85,7 +85,7 @@ public void UpdateRenderedLine(RayStep[] lines, PointerResult result, bool selec
                 }
                 else
                 {
-                    LineBase.LastPoint = RayStep.GetPointByDistance(lines, ExtentOverride.Value);
+                    LineBase.LastPoint = RayStep.GetPointByDistance(lines, extent);
                 }
 
                 if (selectPressed)