Fixup documentation

Author: keveleigh

Assets/MixedRealityToolkit.Providers/OpenVR/OpenVRDeviceManager.cs

@@ -11,7 +11,7 @@
 namespace Microsoft.MixedReality.Toolkit.OpenVR.Input
 {
     /// <summary>
-    /// Manages Open VR Devices using unity's input system.
+    /// Manages Open VR devices using Unity's input system.
     /// </summary>
     [MixedRealityDataProvider(
         typeof(IMixedRealityInputSystem),

Assets/MixedRealityToolkit.Providers/WindowsMixedReality/XR2018/WindowsMixedRealityArticulatedHand.cs

@@ -27,7 +27,7 @@
 namespace Microsoft.MixedReality.Toolkit.WindowsMixedReality.Input
 {
     /// <summary>
-    /// A Windows Mixed Reality Controller Instance.
+    /// A Windows Mixed Reality articulated hand instance.
     /// </summary>
     [MixedRealityController(
         SupportedControllerType.ArticulatedHand,

Assets/MixedRealityToolkit.Providers/WindowsMixedReality/XRSDK/Controllers/BaseWindowsMixedRealityXRSDKSource.cs

@@ -14,7 +14,7 @@
 namespace Microsoft.MixedReality.Toolkit.XRSDK.WindowsMixedReality
 {
     /// <summary>
-    /// A Windows Mixed Reality Source instance.
+    /// A Windows Mixed Reality source instance.
     /// </summary>
     public abstract class BaseWindowsMixedRealityXRSDKSource : GenericXRSDKController
     {

Assets/MixedRealityToolkit.Providers/WindowsMixedReality/XRSDK/Controllers/WindowsMixedRealityXRSDKArticulatedHand.cs

@@ -3,13 +3,13 @@
 
 using Microsoft.MixedReality.Toolkit.Input;
 using Microsoft.MixedReality.Toolkit.Utilities;
-using Microsoft.MixedReality.Toolkit.WindowsMixedReality;
 using System;
 using System.Collections.Generic;
 using UnityEngine;
 using UnityEngine.XR;
 
 #if WINDOWS_UWP
+using Microsoft.MixedReality.Toolkit.WindowsMixedReality;
 using UnityEngine.XR.WindowsMR;
 using Windows.Perception.People;
 using Windows.UI.Input.Spatial;
@@ -18,12 +18,11 @@
 namespace Microsoft.MixedReality.Toolkit.XRSDK.WindowsMixedReality
 {
     /// <summary>
-    /// XR SDK implementation of the Windows Mixed Reality motion controllers.
+    /// XR SDK implementation of Windows Mixed Reality articulated hands.
     /// </summary>
     [MixedRealityController(
         SupportedControllerType.ArticulatedHand,
-        new[] { Handedness.Left, Handedness.Right },
-        "StandardAssets/Textures/MotionController")]
+        new[] { Handedness.Left, Handedness.Right })]
     public class WindowsMixedRealityXRSDKArticulatedHand : BaseWindowsMixedRealityXRSDKSource
     {
         /// <summary>

Assets/MixedRealityToolkit.Providers/WindowsMixedReality/XRSDK/Controllers/WindowsMixedRealityXRSDKMotionController.cs

@@ -7,7 +7,7 @@
 namespace Microsoft.MixedReality.Toolkit.XRSDK.WindowsMixedReality
 {
     /// <summary>
-    /// XR SDK implementation of the Windows Mixed Reality motion controllers.
+    /// XR SDK implementation of Windows Mixed Reality motion controllers.
     /// </summary>
     [MixedRealityController(
         SupportedControllerType.WindowsMixedReality,

Assets/MixedRealityToolkit.Providers/WindowsMixedReality/XRSDK/WindowsMixedRealityDeviceManager.cs

@@ -15,7 +15,7 @@
 namespace Microsoft.MixedReality.Toolkit.XRSDK.WindowsMixedReality
 {
     /// <summary>
-    /// Manages Open VR Devices using unity's input system.
+    /// Manages XR SDK devices on the Windows Mixed Reality platform.
     /// </summary>
     [MixedRealityDataProvider(
         typeof(IMixedRealityInputSystem),

Assets/MixedRealityToolkit.Providers/XRSDK/XRSDKDeviceManager.cs

@@ -11,7 +11,7 @@
 namespace Microsoft.MixedReality.Toolkit.XRSDK.Input
 {
     /// <summary>
-    /// Manages Open VR Devices using unity's input system.
+    /// Manages XR SDK devices.
     /// </summary>
     [MixedRealityDataProvider(
         typeof(IMixedRealityInputSystem),
@@ -35,7 +35,7 @@ public XRSDKDeviceManager(
         /// <inheritdoc />
         public virtual bool CheckCapability(MixedRealityCapability capability)
         {
-            // The OpenVR platform supports motion controllers.
+            // The XR SDK platform supports motion controllers.
             return (capability == MixedRealityCapability.MotionController);
         }
 
@@ -46,6 +46,7 @@ public virtual bool CheckCapability(MixedRealityCapability capability)
         private bool wasLeftInputDeviceValid = false;
         private bool wasRightInputDeviceValid = false;
 
+        /// <inheritdoc/>
         public override void Update()
         {
             base.Update();
@@ -126,9 +127,9 @@ public override void Update()
         #region Controller Utilities
 
         /// <summary>
-        /// Gets or adds a controller using the joystick name provided.
+        /// Gets or adds a controller using the InputDevice name provided.
         /// </summary>
-        /// <param name="joystickName">The name of the joystick from Unity's <see href="https://docs.unity3d.com/ScriptReference/Input.GetJoystickNames.html">Input.GetJoystickNames</see></param>
+        /// <param name="inputDevice">The InputDevice from XR SDK.</param>
         /// <returns>The controller reference.</returns>
         protected virtual GenericXRSDKController GetOrAddController(InputDevice inputDevice)
         {
@@ -188,7 +189,10 @@ protected virtual GenericXRSDKController GetOrAddController(InputDevice inputDev
             return detectedController;
         }
 
-        /// <inheritdoc />
+        /// <summary>
+        /// Gets the current controller type for the InputDevice name provided.
+        /// </summary>
+        /// <param name="inputDevice">The InputDevice from XR SDK.</param>
         protected virtual void RemoveController(InputDevice inputDevice)
         {
             GenericXRSDKController controller = GetOrAddController(inputDevice);
@@ -210,7 +214,7 @@ protected virtual void RemoveController(InputDevice inputDevice)
         /// <summary>
         /// Gets the concrete type of the detected controller, based on the <see cref="SupportedControllerType"/> and defined per-platform.
         /// </summary>
-        /// <param name="supportedControllerType">The current controller type from the enum.</param>
+        /// <param name="supportedControllerType">The current controller type.</param>
         /// <returns>The concrete type of the currently detected controller.</returns>
         protected virtual Type GetControllerType(SupportedControllerType supportedControllerType)
         {
@@ -220,14 +224,18 @@ protected virtual Type GetControllerType(SupportedControllerType supportedContro
         /// <summary>
         /// Returns the <see cref="InputSourceType"/> of the currently detected controller, based on the <see cref="SupportedControllerType"/>.
         /// </summary>
-        /// <param name="supportedControllerType">The current controller type from the enum.</param>
+        /// <param name="supportedControllerType">The current controller type.</param>
         /// <returns>The enum value of the currently detected controller's InputSource type.</returns>
         protected virtual InputSourceType GetInputSourceType(SupportedControllerType supportedControllerType)
         {
             return InputSourceType.Controller;
         }
 
-        /// <inheritdoc />
+        /// <summary>
+        /// Gets the current controller type for the InputDevice name provided.
+        /// </summary>
+        /// <param name="inputDevice">The InputDevice from XR SDK.</param>
+        /// <returns>The supported controller type.</returns>
         protected virtual SupportedControllerType GetCurrentControllerType(InputDevice inputDevice)
         {
             Debug.Log($"{inputDevice.name} does not have a defined controller type, falling back to generic controller type");

Assets/MixedRealityToolkit/Definitions/Devices/MixedRealityControllerMapping.cs

@@ -234,7 +234,7 @@ private MixedRealityInteractionMapping[] GetDefaultInteractionMappings()
         }
 
         /// <summary>
-        /// Synchronizes the Input Actions of the same physical controller of a different concrete type.
+        /// Synchronizes the input actions of the same physical controller of a different concrete type.
         /// </summary>
         internal void SynchronizeInputActions(MixedRealityInteractionMapping[] otherControllerMapping)
         {