Small improvements

Author: arimger

Assets/Editor Toolbox/Editor/ToolboxDefines.cs

@@ -0,0 +1,7 @@
+namespace Toolbox.Editor
+{
+    internal static class ToolboxDefines
+    {
+        internal const string defaultListsDefine = "TOOLBOX_FORCE_DEFAULT_LISTS";
+    }
+}

Assets/Editor Toolbox/Editor/ToolboxDefines.cs.meta

@@ -10,6 +10,11 @@ namespace Toolbox.Editor
 {
     using Toolbox.Editor.Drawers;
 
+    //TODO:
+    //1. dedicated class to initialize and hold drawer-related data
+    //2. dedicated class used for settings initialization
+    //3. separate logic for resettings active drawers
+
     internal static class ToolboxDrawerModule
     {
         [InitializeOnLoadMethod]
@@ -271,14 +276,13 @@ internal static bool HasTargetTypeDrawer(Type type)
 
         internal static void ForceDefaultLists(bool value)
         {
-            const string targetDefine = "TOOLBOX_FORCE_DEFAULT_LISTS";
             if (value)
             {
-                ScriptingUtility.AppendDefine(targetDefine);
+                ScriptingUtility.AppendDefine(ToolboxDefines.defaultListsDefine);
             }
             else
             {
-                ScriptingUtility.RemoveDefine(targetDefine);
+                ScriptingUtility.RemoveDefine(ToolboxDefines.defaultListsDefine);
             }
         }
 
@@ -426,7 +430,7 @@ internal static ToolboxPropertyHandler GetPropertyHandler(SerializedProperty pro
 
         //TODO:
         //NOTE: unfortunately there is no valid, non-reflection way to check if property has a custom native drawer
-        private readonly static MethodInfo getDrawerTypeForTypeMethod =
+        private static readonly MethodInfo getDrawerTypeForTypeMethod =
             ReflectionUtility.GetEditorMethod("UnityEditor.ScriptAttributeUtility", "GetDrawerTypeForType",
                 BindingFlags.NonPublic | BindingFlags.Static);
     }

Assets/Editor Toolbox/Editor/ToolboxDrawerModule.cs

@@ -82,9 +82,10 @@ internal class ToolboxEditorSettings : ScriptableObject, IToolboxGeneralSettings
         [SerializeField, ReorderableList(ListStyle.Boxed)]
         private List<FolderData> customFolders = new List<FolderData>();
 
-        [SerializeField]
+        [SerializeField, Tooltip("Set to false if you don't want to use Toolbox attributes and related features.")]
         private bool useToolboxDrawers = true;
-        [SerializeField, Tooltip("Inspectors will use the default ReorderableList instead of a simple hierarchy.")]
+        [SerializeField, Tooltip("By default, Inspectors will use the built-in version of the list instead of the Toolbox-based one. " +
+            "Keep in mind that built-in properties don't support Toolbox attributes. \n\n Changing this property will recompile the code.")]
         private bool forceDefaultLists;
 
         [SerializeField, ReorderableList(ListStyle.Boxed), ClassExtends(typeof(ToolboxDecoratorDrawer<>))]

Assets/Editor Toolbox/Editor/ToolboxEditorSettings.cs

@@ -262,7 +262,9 @@ private void DrawInspectorSettings()
             EditorGUI.BeginChangeCheck();
             EditorGUILayout.Space();
             EditorGUILayout.PropertyField(useToolboxDrawersProperty);
+            EditorGUI.BeginDisabledGroup(!useToolboxDrawersProperty.boolValue);
             EditorGUILayout.PropertyField(forceDefaultListsProperty);
+            EditorGUI.EndDisabledGroup();
             EditorGUILayout.Space();
 
             var validateInspector = false;