fix: minor bug that caused quick settings panel to remain visible even after all probes were destroyed

Author: dbirman

Assets/Scripts/TrajectoryPlanner/TP_ProbeQuickSettings.cs

@@ -96,9 +96,17 @@ public void UpdateCoordinates()
 
         public void UpdateQuickUI()
         {
-            _probeIdText.text = ProbeManager.ActiveProbeManager.name;
-            _probeIdText.color = ProbeManager.ActiveProbeManager.Color;
-            SetColorChooserColor(ProbeManager.ActiveProbeManager.Color);
+            if (ProbeManager.ActiveProbeManager != null)
+            {
+                _probeIdText.text = ProbeManager.ActiveProbeManager.name;
+                _probeIdText.color = ProbeManager.ActiveProbeManager.Color;
+                SetColorChooserColor(ProbeManager.ActiveProbeManager.Color);
+            }
+            else
+            {
+                _probeIdText.text = "";
+                UpdateInteractable(true);
+            }
         }
 
         /// <summary>

Assets/Scripts/TrajectoryPlanner/TrajectoryPlannerManager.cs

@@ -346,7 +346,7 @@ public void DestroyProbe(ProbeManager probeManager)
             Destroy(probeManager.gameObject);
 
             // Cleanup UI if this was last probe in scene
-            var realProbes = ProbeManager.Instances.Where(x => x.ProbeType != ProbeProperties.ProbeType.Placeholder);
+            var realProbes = ProbeManager.Instances.Where(x => x.ProbeType != ProbeProperties.ProbeType.Placeholder && x != probeManager);
 
             if (realProbes.Count() > 0)
             {
@@ -366,6 +366,7 @@ public void DestroyProbe(ProbeManager probeManager)
                 _probeQuickSettings.UpdateInteractable(true);
                 SetSurfaceDebugActive(false);
                 UpdateQuickSettings();
+                UpdateQuickSettingsProbeIdText();
             }
 
             _probeAddedOrRemovedEvent.Invoke();