Refactor: ReflectionProbe inspector is now auto-generated

Source/EditorManaged/Inspectors/ReflectionProbeInspector.cs

@@ -15,106 +15,21 @@ namespace bs.Editor
     [CustomInspector(typeof(ReflectionProbe))]
     internal class ReflectionProbeInspector : Inspector
     {
-        private GUIEnumField probeTypeField = new GUIEnumField(typeof(ReflectionProbeType), new LocEdString("Probe type"));
-        private GUIVector3Field extentsField = new GUIVector3Field(new LocEdString("Extents"));
-        private GUIFloatField radiusField = new GUIFloatField(new LocEdString("Radius"));
-        private GUITextureField customTextureField = new GUITextureField(new LocEdString("Custom texture"));
         private GUIButton captureButton = new GUIButton(new LocEdString("Capture"));
 
-        private InspectableState modifyState;
-
         /// <inheritdoc/>
         protected internal override void Initialize()
         {
-            if (InspectedObject != null)
-            {
-                ReflectionProbe probe = (ReflectionProbe)InspectedObject;
-
-                probeTypeField.OnSelectionChanged += x =>
-                {
-                    probe.Type = (ReflectionProbeType)x;
-
-                    ToggleTypeSpecificFields((ReflectionProbeType) x);
-                };
-
-                radiusField.OnChanged += x => { probe.Radius = x; MarkAsModified(); };
-                radiusField.OnConfirmed += ConfirmModify;
-                radiusField.OnFocusLost += ConfirmModify;
-
-                extentsField.OnValueChanged += x => { probe.Extents = x; MarkAsModified(); };
-                extentsField.OnConfirm += x => ConfirmModify();
-                extentsField.OnFocusLost += ConfirmModify;
-
-                customTextureField.OnChanged += x =>
-                {
-                    probe.CustomTexture = x.As<Texture>();
-                    MarkAsModified();
-                    ConfirmModify();
-                };
-
-                captureButton.OnClick += () => probe.Capture();
-
-                Layout.AddElement(probeTypeField);
-                Layout.AddElement(radiusField);
-                Layout.AddElement(extentsField);
-                Layout.AddElement(customTextureField);
-                Layout.AddSpace(10);
-                Layout.AddElement(captureButton);
-
-                ToggleTypeSpecificFields(probe.Type);
-            }
-        }
-
-        /// <inheritdoc/>
-        protected internal override InspectableState Refresh(bool force = false)
-        {
-            ReflectionProbe probe = InspectedObject as ReflectionProbe;
-            if (probe == null)
-                return InspectableState.NotModified;
-
-            ReflectionProbeType probeType = probe.Type;
-            if (probeTypeField.Value != (ulong)probeType)
-                ToggleTypeSpecificFields(probeType);
-
-            probeTypeField.Value = (ulong)probeType;
-            radiusField.Value = probe.Radius;
-            extentsField.Value = probe.Extents;
-            customTextureField.TextureRef = probe.CustomTexture;
-
-            InspectableState oldState = modifyState;
-            if (modifyState.HasFlag(InspectableState.Modified))
-                modifyState = InspectableState.NotModified;
-
-            return oldState;
-        }
-
-        /// <summary>
-        /// Enables or disables different GUI elements depending on the probe type.
-        /// </summary>
-        /// <param name="type">Probe type to show GUI elements for.</param>
-        private void ToggleTypeSpecificFields(ReflectionProbeType type)
-        {
-            bool isBox = type == ReflectionProbeType.Box;
+            ReflectionProbe probe = (ReflectionProbe)InspectedObject;
 
-            radiusField.Active = !isBox;
-            extentsField.Active = isBox;
-        }
+            drawer.AddDefault(probe);
+            drawer.AddConditional("Radius", () => probe.Type == ReflectionProbeType.Sphere);
+            drawer.AddConditional("Extents", () => probe.Type == ReflectionProbeType.Box);
 
-        /// <summary>
-        /// Marks the contents of the inspector as modified.
-        /// </summary>
-        protected void MarkAsModified()
-        {
-            modifyState |= InspectableState.ModifyInProgress;
-        }
+            captureButton.OnClick += () => probe.Capture();
 
-        /// <summary>
-        /// Confirms any queued modifications.
-        /// </summary>
-        protected void ConfirmModify()
-        {
-            if (modifyState.HasFlag(InspectableState.ModifyInProgress))
-                modifyState |= InspectableState.Modified;
+            Layout.AddSpace(10);
+            Layout.AddElement(captureButton);
         }
     }
 

Source/bsf

@@ -1 +1 @@
-Subproject commit 298bdb6692b2d048c59851e27869e6c0be10f0f4
+Subproject commit 0badff48f88f501d8de865b438aeb3d494337e79