Finish GameObject Inspector, some UI cleanups

Author: sinai-dev

src/UI/Inspectors/GameObjectInspector.cs

@@ -0,0 +1,53 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using UnityEngine;
+using UnityEngine.UI;
+using UnityExplorer.UI.Widgets;
+
+namespace UnityExplorer.UI.Inspectors
+{
+    public class ComponentCell : ButtonCell
+    {
+        public Toggle BehaviourToggle;
+        public ButtonRef DestroyButton;
+
+        public Action<bool, int> OnBehaviourToggled;
+        public Action<int> OnDestroyClicked;
+
+        private void BehaviourToggled(bool val)
+        {
+            OnBehaviourToggled?.Invoke(val, CurrentDataIndex);
+        }
+
+        private void DestroyClicked()
+        {
+            OnDestroyClicked?.Invoke(CurrentDataIndex);
+        }
+
+        public override GameObject CreateContent(GameObject parent)
+        {
+            var root = base.CreateContent(parent);
+
+            // Add mask to button so text doesnt overlap on Close button
+            this.Button.Component.gameObject.AddComponent<Mask>().showMaskGraphic = true;
+
+            // Behaviour toggle
+
+            var toggleObj = UIFactory.CreateToggle(UIRoot, "BehaviourToggle", out BehaviourToggle, out var behavText);
+            UIFactory.SetLayoutElement(toggleObj, minHeight: 25, minWidth: 25);
+            BehaviourToggle.onValueChanged.AddListener(BehaviourToggled);
+            // put at first object
+            toggleObj.transform.SetSiblingIndex(0);
+
+            // Destroy button
+
+            DestroyButton = UIFactory.CreateButton(UIRoot, "DestroyButton", "X", new Color(0.3f, 0.2f, 0.2f));
+            UIFactory.SetLayoutElement(DestroyButton.Component.gameObject, minHeight: 21, minWidth: 25);
+            DestroyButton.OnClick += DestroyClicked;
+
+            return root;
+        }
+    }
+}

src/UI/Inspectors/GameObjectWidgets/ComponentCell.cs

@@ -0,0 +1,116 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using UnityEngine;
+using UnityExplorer.UI.Widgets;
+
+namespace UnityExplorer.UI.Inspectors
+{
+    public class ComponentList : ButtonListHandler<Component, ComponentCell>
+    {
+        public GameObjectInspector Parent;
+
+        public ComponentList(ScrollPool<ComponentCell> scrollPool, Func<List<Component>> getEntriesMethod) 
+            : base(scrollPool, getEntriesMethod, null, null, null)
+        {
+            base.SetICell = SetComponentCell;
+            base.ShouldDisplay = CheckShouldDisplay;
+            base.OnCellClicked = OnComponentClicked;
+        }
+
+        private bool CheckShouldDisplay(Component _, string __) => true;
+
+        public override void OnCellBorrowed(ComponentCell cell)
+        {
+            base.OnCellBorrowed(cell);
+
+            cell.OnBehaviourToggled += OnBehaviourToggled;
+            cell.OnDestroyClicked += OnDestroyClicked;
+        }
+
+        public override void SetCell(ComponentCell cell, int index)
+        {
+            base.SetCell(cell, index);
+        }
+
+        private void OnComponentClicked(int index)
+        {
+            var entries = GetEntries();
+
+            if (index < 0 || index >= entries.Count)
+                return;
+
+            var comp = entries[index];
+            if (comp)
+                InspectorManager.Inspect(comp);
+        }
+
+        private void OnBehaviourToggled(bool value, int index)
+        {
+            try
+            {
+                var entries = GetEntries();
+                var comp = entries[index];
+
+                if (comp.TryCast<Behaviour>() is Behaviour behaviour)
+                    behaviour.enabled = value;
+            }
+            catch (Exception ex)
+            {
+                ExplorerCore.LogWarning($"Exception toggling Behaviour.enabled: {ex.ReflectionExToString()}");
+            }
+        }
+
+        private void OnDestroyClicked(int index)
+        {
+            try
+            {
+                var entries = GetEntries();
+                var comp = entries[index];
+
+                GameObject.DestroyImmediate(comp);
+
+                Parent.UpdateComponents();
+            }
+            catch (Exception ex)
+            {
+                ExplorerCore.LogWarning($"Exception destroying Component: {ex.ReflectionExToString()}");
+            }
+        }
+
+        private static readonly Dictionary<string, string> compToStringCache = new Dictionary<string, string>();
+
+        // Called from ButtonListHandler.SetCell, will be valid
+        private void SetComponentCell(ComponentCell cell, int index)
+        {
+            var entries = GetEntries();
+            cell.Enable();
+
+            var comp = entries[index];
+            var type = comp.GetActualType();
+
+            if (!compToStringCache.ContainsKey(type.AssemblyQualifiedName))
+                compToStringCache.Add(type.AssemblyQualifiedName, SignatureHighlighter.Parse(type, true));
+
+            cell.Button.ButtonText.text = compToStringCache[type.AssemblyQualifiedName];
+
+            if (typeof(Behaviour).IsAssignableFrom(type))
+            {
+                cell.BehaviourToggle.interactable = true;
+                cell.BehaviourToggle.Set(comp.TryCast<Behaviour>().enabled, false);
+            }
+            else
+            {
+                cell.BehaviourToggle.interactable = false;
+                cell.BehaviourToggle.Set(false, false);
+            }
+
+            // if component is the first index it must be the transform, dont show Destroy button for it.
+            if (index == 0 && cell.DestroyButton.Component.gameObject.activeSelf)
+                cell.DestroyButton.Component.gameObject.SetActive(false);
+            else if (index > 0 && !cell.DestroyButton.Component.gameObject.activeSelf)
+                cell.DestroyButton.Component.gameObject.SetActive(true);
+        }
+    }
+}