Possibility to force drawing default ReorderableLists

Author: arimger

Assets/Editor Toolbox/Editor/ToolboxDrawerModule.cs

@@ -244,6 +244,7 @@ internal static void UpdateDrawers(IToolboxInspectorSettings settings)
             //create all type-only-related drawers
             PrepareTargetTypeDrawers(settings);
 
+            ForceDefaultLists(settings.ForceDefaultLists);
             //log errors into console only once
             validationEnabled = false;
         }
@@ -268,6 +269,19 @@ internal static bool HasTargetTypeDrawer(Type type)
             return targetTypeDrawers.ContainsKey(type.IsGenericType ? type.GetGenericTypeDefinition() : type);
         }
 
+        internal static void ForceDefaultLists(bool value)
+        {
+            const string targetDefine = "TOOLBOX_FORCE_DEFAULT_LISTS";
+            if (value)
+            {
+                ScriptingUtility.AppendDefine(targetDefine);
+            }
+            else
+            {
+                ScriptingUtility.RemoveDefine(targetDefine);
+            }
+        }
+
         internal static ToolboxDecoratorDrawerBase GetDecoratorDrawer<T>(T attribute) where T : ToolboxDecoratorAttribute
         {
             return GetDecoratorDrawer(attribute.GetType());

Assets/Editor Toolbox/Editor/ToolboxEditorSettings.cs

@@ -43,6 +43,7 @@ internal interface IToolboxInspectorSettings
         void SetAllPossibleTargetTypeDrawers();
 
         bool UseToolboxDrawers { get; }
+        bool ForceDefaultLists { get; }
 
         List<SerializedType> DecoratorDrawerHandlers { get; }
         List<SerializedType> ConditionDrawerHandlers { get; }
@@ -83,6 +84,8 @@ internal class ToolboxEditorSettings : ScriptableObject, IToolboxGeneralSettings
 
         [SerializeField]
         private bool useToolboxDrawers = true;
+        [SerializeField, Tooltip("Inspectors will use the default ReorderableList instead of a simple hierarchy.")]
+        private bool forceDefaultLists;
 
         [SerializeField, ReorderableList(ListStyle.Boxed), ClassExtends(typeof(ToolboxDecoratorDrawer<>))]
         private List<SerializedType> decoratorDrawerHandlers = new List<SerializedType>();
@@ -327,6 +330,12 @@ public bool UseToolboxDrawers
             set => useToolboxDrawers = value;
         }
 
+        public bool ForceDefaultLists 
+        { 
+            get => forceDefaultLists; 
+            set => forceDefaultLists = value; 
+        }
+
         public List<SerializedType> DecoratorDrawerHandlers
         {
             get => decoratorDrawerHandlers;
@@ -357,7 +366,6 @@ public List<SerializedType> TargetTypeDrawerHandlers
             set => targetTypeDrawerHandlers = value;
         }
 
-
         private static class Defaults
         {
             internal const float largeFolderIconScaleDefault = 0.8f;

Assets/Editor Toolbox/Editor/ToolboxEditorSettingsEditor.cs

@@ -26,6 +26,7 @@ internal class ToolboxEditorSettingsEditor : ToolboxEditor
         private SerializedProperty showSelectionsCountProperty;
         private SerializedProperty useToolboxFoldersProperty;
         private SerializedProperty useToolboxDrawersProperty;
+        private SerializedProperty forceDefaultListsProperty;
         private SerializedProperty largeIconScaleProperty;
         private SerializedProperty smallIconScaleProperty;
         private SerializedProperty largeIconPaddingProperty;
@@ -86,6 +87,7 @@ private void OnEnable()
 #endif
             //inspector-related properties
             useToolboxDrawersProperty = serializedObject.FindProperty("useToolboxDrawers");
+            forceDefaultListsProperty = serializedObject.FindProperty("forceDefaultLists");
 
             drawerHandlersLists = new ReorderableListBase[5];
 #if UNITY_2019_3_OR_NEWER
@@ -260,6 +262,7 @@ private void DrawInspectorSettings()
             EditorGUI.BeginChangeCheck();
             EditorGUILayout.Space();
             EditorGUILayout.PropertyField(useToolboxDrawersProperty);
+            EditorGUILayout.PropertyField(forceDefaultListsProperty);
             EditorGUILayout.Space();
 
             var validateInspector = false;

Assets/Editor Toolbox/Editor/ToolboxPropertyHandler.cs

@@ -444,15 +444,23 @@ public void OnGuiDefault(SerializedProperty property)
         /// </summary>
         public void OnGuiDefault(SerializedProperty property, GUIContent label)
         {
+#if TOOLBOX_FORCE_DEFAULT_LISTS
+            if (isArray)
+            {
+                EditorGUILayout.PropertyField(property, label, true);
+                return;
+            }
+#endif
             //all "single" properties and native drawers should be drawn in the native way
             if (hasBuiltInPropertyDrawer)
             {
                 ToolboxEditorGui.DrawNativeProperty(property, label);
-                return;
             }
-
             //handles property in default native way but supports ToolboxDrawers in children
-            ToolboxEditorGui.DrawDefaultProperty(property, label);
+            else
+            {
+                ToolboxEditorGui.DrawDefaultProperty(property, label);
+            }
         }
     }
 }

Assets/Editor Toolbox/Editor/Utilities/ScriptingUtility.cs

@@ -0,0 +1,45 @@
+using System.Collections.Generic;
+using System.Linq;
+
+using UnityEditor;
+
+namespace Toolbox.Editor
+{
+    public static class ScriptingUtility
+    {
+        public static List<string> GetDefines()
+        {
+            var defines = PlayerSettings.GetScriptingDefineSymbolsForGroup(EditorUserBuildSettings.selectedBuildTargetGroup);
+            return defines.Split(';').ToList();
+        }
+
+        public static void SetDefines(List<string> definesList)
+        {
+            var defines = string.Join(";", definesList.ToArray());
+            PlayerSettings.SetScriptingDefineSymbolsForGroup(EditorUserBuildSettings.selectedBuildTargetGroup, defines);
+        }
+
+        public static void AppendDefine(string define)
+        {
+            var definesList = GetDefines();
+            if (definesList.Contains(define))
+            {
+                return;
+            }
+
+            definesList.Add(define);
+            SetDefines(definesList);
+        }
+
+        public static void RemoveDefine(string define)
+        {
+            var definesList = GetDefines();
+            if (definesList.RemoveAll(s => s == define) == 0)
+            {
+                return;
+            }
+
+            SetDefines(definesList);
+        }
+    }
+}

Assets/Editor Toolbox/Editor/Utilities/ScriptingUtility.cs.meta

@@ -47,6 +47,7 @@ MonoBehaviour:
     smallIcon: {fileID: 2800000, guid: e99d63b63394eba4aaf10385704d0ef3, type: 3}
     iconName: 
   useToolboxDrawers: 1
+  forceDefaultLists: 0
   decoratorDrawerHandlers:
   - typeReference: Toolbox.Editor.Drawers.BeginGroupAttributeDrawer, Toolbox.Editor
   - typeReference: Toolbox.Editor.Drawers.BeginHorizontalAttributeDrawer, Toolbox.Editor