Minor refactor changes

Author: arimger

Assets/Editor Toolbox/Editor/IToolboxEditorDrawer.cs

@@ -4,17 +4,14 @@ namespace Toolbox.Editor
 {
     public interface IToolboxEditorDrawer
     {
-        /// <summary>
-        /// Draws each available property using internally <see cref="Drawers.ToolboxDrawer"/>s.
-        /// </summary>
         void DrawEditor(SerializedObject serializedObject);
         /// <summary>
         /// Draws <see cref="SerializedProperty"/>/ies in the default (native) way.
         /// </summary>
         /// <param name="serializedObject"></param>
         void DrawDefaultEditor(SerializedObject serializedObject);
         /// <summary>
-        /// TODO
+        /// Draws each available property using internally <see cref="Drawers.ToolboxDrawer"/>s.
         /// </summary>
         /// <param name="serializedObject"></param>
         void DrawToolboxEditor(SerializedObject serializedObject);
@@ -26,5 +23,7 @@ public interface IToolboxEditorDrawer
         /// Forces provided <see cref="SerializedProperty"/> to be ignored in the drawing process.
         /// </summary>
         void IgnoreProperty(string propertyPath);
+        bool IsPropertyIgnored(SerializedProperty property);
+        bool IsPropertyIgnored(string propertyPath);
     }
 }

Assets/Editor Toolbox/Editor/ToolboxEditor.cs

@@ -22,12 +22,10 @@ public sealed override void OnInspectorGUI()
             ToolboxEditorHandler.HandleToolboxEditor(this);
         }
 
-        //TODO: how to handle this method, currently unused
         /// <inheritdoc />
+        [Obsolete("To draw properties in a different way override the Drawer property.")]
         public virtual void DrawCustomProperty(SerializedProperty property)
-        {
-            //Drawer.DrawProperty(property);
-        }
+        { }
 
         /// <inheritdoc />
         public virtual void DrawCustomInspector()
@@ -49,7 +47,7 @@ public void IgnoreProperty(string propertyPath)
 
 
         Editor IToolboxEditor.ContextEditor => this;
-        public IToolboxEditorDrawer Drawer { get; } = new ToolboxEditorDrawer();
+        public virtual IToolboxEditorDrawer Drawer { get; } = new ToolboxEditorDrawer();
 
 #pragma warning disable 0067
         [Obsolete("ToolboxEditorHandler.OnBeginToolboxEditor")]

Assets/Editor Toolbox/Editor/ToolboxEditorDrawer.cs

@@ -5,17 +5,23 @@
 
 namespace Toolbox.Editor
 {
+    /// <summary>
+    /// Default drawer responsible for drawing <see cref="UnityEditor.Editor"/>s.
+    /// This helper class that can be used across many Toolbox-based editors that cannot share the same base class.
+    /// </summary>
     public class ToolboxEditorDrawer : IToolboxEditorDrawer
     {
         private readonly HashSet<string> propertiesToIgnore = new HashSet<string>();
-        //TODO:
         private readonly Action<SerializedProperty> toolboxDrawingAction;
         private readonly Action<SerializedProperty> defaultDrawingAction;
 
-        public ToolboxEditorDrawer()
+        public ToolboxEditorDrawer() : this(ToolboxEditorGui.DrawToolboxProperty, ToolboxEditorGui.DrawNativeProperty)
+        { }
+
+        public ToolboxEditorDrawer(Action<SerializedProperty> toolboxDrawingAction, Action<SerializedProperty> defaultDrawingAction)
         {
-            toolboxDrawingAction = ToolboxEditorGui.DrawToolboxProperty;
-            defaultDrawingAction = ToolboxEditorGui.DrawNativeProperty;
+            this.toolboxDrawingAction = toolboxDrawingAction;
+            this.defaultDrawingAction = defaultDrawingAction;
         }
 
         private void DrawProperty(SerializedProperty property, Action<SerializedProperty> drawingAction)
@@ -67,13 +73,13 @@ public void DrawEditor(SerializedObject serializedObject, Action<SerializedPrope
         /// <inheritdoc />
         public void DrawToolboxEditor(SerializedObject serializedObject)
         {
-            DrawEditor(serializedObject, ToolboxEditorGui.DrawToolboxProperty);
+            DrawEditor(serializedObject, toolboxDrawingAction);
         }
 
         /// <inheritdoc />
         public void DrawDefaultEditor(SerializedObject serializedObject)
         {
-            DrawEditor(serializedObject, ToolboxEditorGui.DrawNativeProperty);
+            DrawEditor(serializedObject, defaultDrawingAction);
         }
 
         /// <inheritdoc />
@@ -88,6 +94,12 @@ public void IgnoreProperty(string propertyPath)
             propertiesToIgnore.Add(propertyPath);
         }
 
+        /// <inheritdoc />
+        public bool IsPropertyIgnored(SerializedProperty property)
+        {
+            return IsPropertyIgnored(property.propertyPath);
+        }
+
         /// <inheritdoc />
         public bool IsPropertyIgnored(string propertyPath)
         {

Assets/Editor Toolbox/Editor/ToolboxEditorGui.cs

@@ -546,7 +546,7 @@ public static void DrawNativeProperty(SerializedProperty property)
         /// </summary>
         public static void DrawNativeProperty(SerializedProperty property, GUIContent label)
         {
-            EditorGUILayout.PropertyField(property, label, property.isExpanded);
+            EditorGUILayout.PropertyField(property, label);
         }
 
         /// <summary>