added missing license header, separated get / set in bounds control, added missing braces and fixed some whitespace issues

Author: bethalha-havok

Assets/MixedRealityToolkit.SDK/Experimental/Features/UX/BoundsControl/BoundsControl.cs

@@ -131,7 +131,11 @@ public BoundsControlActivationType BoundsControlActivation
         /// <summary>
         /// Check to draw a tether point from the handles to the hand when manipulating.
         /// </summary>
-        public bool DrawTetherWhenManipulating { get => drawTetherWhenManipulating; set => drawTetherWhenManipulating = value;}
+        public bool DrawTetherWhenManipulating
+        {
+            get => drawTetherWhenManipulating;
+            set => drawTetherWhenManipulating = value;
+        }
 
         [SerializeField]
         [Tooltip("Add a Collider here if you do not want the handle colliders to interact with another object's collider.")]
@@ -140,7 +144,11 @@ public BoundsControlActivationType BoundsControlActivation
         /// <summary>
         /// Add a Collider here if you do not want the handle colliders to interact with another object's collider.
         /// </summary>
-        public Collider HandlesIgnoreCollider { get => handlesIgnoreCollider; set => handlesIgnoreCollider = value; }
+        public Collider HandlesIgnoreCollider
+        {
+            get => handlesIgnoreCollider;
+            set => handlesIgnoreCollider = value;
+        }
 
         [SerializeField]
         [Tooltip("Flatten bounds in the specified axis or flatten the smallest one if 'auto' is selected")]
@@ -207,47 +215,71 @@ public Vector3 BoxPadding
         /// <summary>
         /// Bounds control box display configuration section.
         /// </summary>
-        public BoxDisplayConfiguration BoxDisplayConfiguration { get => boxDisplayConfiguration; set => boxDisplayConfiguration = value; }
+        public BoxDisplayConfiguration BoxDisplayConfiguration
+        {
+            get => boxDisplayConfiguration;
+            set => boxDisplayConfiguration = value;
+        }
 
         [SerializeField]
         [Tooltip("This section defines the links / lines that are drawn between the corners of the control.")]
         private LinksConfiguration linksConfiguration;
         /// <summary>
         /// This section defines the links / lines that are drawn between the corners of the control.
         /// </summary>
-        public LinksConfiguration LinksConfiguration { get => linksConfiguration; set => linksConfiguration = value; }
+        public LinksConfiguration LinksConfiguration
+        {
+            get => linksConfiguration;
+            set => linksConfiguration = value;
+        }
 
         [SerializeField]
         [Tooltip("Configuration of the scale handles.")]
         private ScaleHandlesConfiguration scaleHandlesConfiguration;
         /// <summary>
         /// Configuration of the scale handles.
         /// </summary>
-        public ScaleHandlesConfiguration ScaleHandlesConfiguration { get => scaleHandlesConfiguration; set => scaleHandlesConfiguration = value; }
+        public ScaleHandlesConfiguration ScaleHandlesConfiguration
+        {
+            get => scaleHandlesConfiguration;
+            set => scaleHandlesConfiguration = value;
+        }
 
         [SerializeField]
         [Tooltip("Configuration of the rotation handles.")]
         private RotationHandlesConfiguration rotationHandlesConfiguration;
         /// <summary>
         /// Configuration of the rotation handles.
         /// </summary>
-        public RotationHandlesConfiguration RotationHandles { get => rotationHandlesConfiguration; set => rotationHandlesConfiguration = value; }
+        public RotationHandlesConfiguration RotationHandles
+        {
+            get => rotationHandlesConfiguration;
+            set => rotationHandlesConfiguration = value;
+        }
 
         [SerializeField]
         [Tooltip("Configuration for Proximity Effect to scale handles or change materials on proximity.")]
         private ProximityEffectConfiguration handleProximityEffectConfiguration;
         /// <summary>
         /// Configuration for Proximity Effect to scale handles or change materials on proximity.
         /// </summary>
-        public ProximityEffectConfiguration HandleProximityEffectConfiguration { get => handleProximityEffectConfiguration; set => handleProximityEffectConfiguration = value; }
+        public ProximityEffectConfiguration HandleProximityEffectConfiguration
+        {
+            get => handleProximityEffectConfiguration;
+            set => handleProximityEffectConfiguration = value;
+        }
 
         [Header("Debug")]
         [Tooltip("Debug only. Component used to display debug messages.")]
         private TextMesh debugText;
         /// <summary>
         /// Component used to display debug messages.
         /// </summary>
-        public TextMesh DebugText { get => debugText; set => debugText = value; }
+        public TextMesh DebugText
+        {
+            get => debugText;
+            set => debugText = value;
+        }
 
         [SerializeField]
         [Tooltip("Determines whether to hide GameObjects (i.e handles, links etc) created and managed by this component in the editor")]
@@ -276,31 +308,47 @@ public bool HideElementsInInspector
         /// <summary>
         /// Event that gets fired when interaction with a rotation handle starts.
         /// </summary>
-        public UnityEvent RotateStarted { get => rotateStarted; set => rotateStarted = value; }
+        public UnityEvent RotateStarted
+        {
+            get => rotateStarted;
+            set => rotateStarted = value;
+        }
 
         [SerializeField]
         [Tooltip("Event that gets fired when interaction with a rotation handle stops.")]
         private UnityEvent rotateStopped = new UnityEvent();
         /// <summary>
         /// Event that gets fired when interaction with a rotation handle stops.
         /// </summary>
-        public UnityEvent RotateStopped { get => rotateStopped; set => rotateStopped = value; }
+        public UnityEvent RotateStopped
+        {
+            get => rotateStopped;
+            set => rotateStopped = value;
+        }
 
         [SerializeField]
         [Tooltip("Event that gets fired when interaction with a scale handle starts.")]
         private UnityEvent scaleStarted = new UnityEvent();
         /// <summary>
         /// Event that gets fired when interaction with a scale handle starts.
         /// </summary>
-        public UnityEvent ScaleStarted { get => scaleStarted; set => scaleStarted = value; }
+        public UnityEvent ScaleStarted
+        {
+            get => scaleStarted;
+            set => scaleStarted = value;
+        }
 
         [SerializeField]
         [Tooltip("Event that gets fired when interaction with a scale handle stops.")]
         private UnityEvent scaleStopped = new UnityEvent();
         /// <summary>
         /// Event that gets fired when interaction with a scale handle stops.
         /// </summary>
-        public UnityEvent ScaleStopped { get => scaleStopped; set => scaleStopped = value; }
+        public UnityEvent ScaleStopped
+        {
+            get => scaleStopped;
+            set => scaleStopped = value;
+        }
 
         #endregion Serialized Fields
 
@@ -866,7 +914,9 @@ private void DestroyRig()
         private void UpdateRigVisibilityInInspector()
         {
             if (!IsInitialized())
+            {
                 return;
+            }
 
             HideFlags desiredFlags = hideElementsInInspector ? HideFlags.HideInHierarchy | HideFlags.HideInInspector : HideFlags.None;
             scaleHandles.UpdateVisibilityInInspector(desiredFlags);

Assets/MixedRealityToolkit.SDK/Features/UX/Scripts/BoundingBox/IBoundsTargetProvider.cs

@@ -1,4 +1,6 @@
-
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT License. See LICENSE in the project root for license information.
+
 using UnityEngine;
 
 namespace Microsoft.MixedReality.Toolkit.UI
@@ -9,7 +11,6 @@ namespace Microsoft.MixedReality.Toolkit.UI
     /// </summary>
     internal interface IBoundsTargetProvider
     {
-
         /// <summary>
         /// Indicates if the provider is currently active
         /// </summary>

Assets/MixedRealityToolkit.Tests/EditModeTests/Experimental/BoundsControlTests.cs

@@ -11,8 +11,7 @@ namespace Microsoft.MixedReality.Toolkit.Tests.Experimental
     /// Tests for edit mode behavior of bounds control
     /// </summary>
     public class BoundsControlTests
-    {
-        
+    {       
         [Test]
         /// <summary>
         /// Tests configuring every property of bounds control in edit mode