Merge pull request #107 from arimger/develop

Develop - 0.12.11

Author: arimger

Assets/Editor Toolbox/CHANGELOG.md

@@ -1,4 +1,11 @@
-## 0.12.0 [28.03.2024]
+## 0.12.11 [05.04.2024]
+
+### Changed:
+- Fix handling the HasNativeTypeDrawer method in Unity 2022.3.23
+- Ignore `m_SerializedDataModeController` in Toolbox wizards
+- Change [Begin/EndHorizontal] behaviour (more customization options to handle available space)
+
+## 0.12.10 [28.03.2024]
 
 ### Changed:
 - Fix fetching and initializing Toolbox features when assets are updating

Assets/Editor Toolbox/Editor/Drawers/Toolbox/Decorator/BeginHorizontalAttributeDrawer.cs

@@ -5,14 +5,25 @@ namespace Toolbox.Editor.Drawers
 {
     public class BeginHorizontalAttributeDrawer : ToolboxDecoratorDrawer<BeginHorizontalAttribute>
     {
+        private static float lastFetchedWidth = 0.0f;
+
         protected override void OnGuiBeginSafe(BeginHorizontalAttribute attribute)
         {
-            var width = EditorGUIUtility.currentViewWidth;
-            //set a new width value for label/field controls
-            EditorGUIUtility.labelWidth = width * attribute.LabelToWidthRatio;
-            EditorGUIUtility.fieldWidth = width * attribute.FieldToWidthRatio;
+            if (GuiLayoutUtility.TryGetLayoutWidth(out var layoutWidth))
+            {
+                lastFetchedWidth = layoutWidth;
+            }
+
+            EditorGUIUtility.labelWidth = attribute.LabelWidth;
+            if (attribute.ControlFieldWidth && attribute.ElementsInLayout > 0)
+            {
+                var width = lastFetchedWidth;
+                width -= attribute.WidthOffset;
+                width -= (attribute.LabelWidth + attribute.WidthOffsetPerElement + EditorGUIUtility.standardVerticalSpacing * 4) * attribute.ElementsInLayout;
+                width /= attribute.ElementsInLayout;
+                EditorGUIUtility.fieldWidth = width;
+            }
 
-            //begin horizontal group using internal utility
             ToolboxLayoutHandler.BeginHorizontal();
         }
     }

Assets/Editor Toolbox/Editor/Drawers/Toolbox/Decorator/BeginHorizontalGroupAttributeDrawer.cs

@@ -18,6 +18,7 @@ static BeginHorizontalGroupAttributeDrawer()
         /// </summary>
         private static readonly ControlDataStorage<Vector2> storage;
 
+        private static float lastFetchedWidth = 0.0f;
 
         private void HandleScrollView(float fixedHeight)
         {
@@ -29,25 +30,33 @@ private void HandleScrollView(float fixedHeight)
             storage.AppendItem(controlId, newScroll);
         }
 
-
         protected override void OnGuiBeginSafe(BeginHorizontalGroupAttribute attribute)
         {
-            var fixedWidth = EditorGUIUtility.currentViewWidth;
-            var fixedHeight = attribute.Height;
-            EditorGUIUtility.labelWidth = fixedWidth * attribute.LabelToWidthRatio;
-            EditorGUIUtility.fieldWidth = fixedWidth * attribute.FieldToWidthRatio;
+            if (GuiLayoutUtility.TryGetLayoutWidth(out var layoutWidth))
+            {
+                lastFetchedWidth = layoutWidth;
+            }
 
             ToolboxLayoutHandler.BeginVertical(Style.groupBackgroundStyle);
             if (attribute.HasLabel)
             {
                 GUILayout.Label(attribute.Label, EditorStyles.boldLabel);
             }
 
-            HandleScrollView(fixedHeight);
+            EditorGUIUtility.labelWidth = attribute.LabelWidth;
+            if (attribute.ControlFieldWidth && attribute.ElementsInLayout > 0)
+            {
+                var width = lastFetchedWidth;
+                width -= attribute.WidthOffset;
+                width -= (attribute.LabelWidth + attribute.WidthOffsetPerElement + EditorGUIUtility.standardVerticalSpacing * 4) * attribute.ElementsInLayout;
+                width /= attribute.ElementsInLayout;
+                EditorGUIUtility.fieldWidth = width;
+            }
+
+            HandleScrollView(attribute.Height);
             ToolboxLayoutHandler.BeginHorizontal();
         }
 
-
         private static class Style
         {
             internal static readonly GUIStyle groupBackgroundStyle;

Assets/Editor Toolbox/Editor/Drawers/Toolbox/PropertyList/ReorderableListExposedAttributeDrawer.cs

@@ -31,7 +31,6 @@ static ReorderableListExposedAttributeDrawer()
 
         private static readonly PropertyDataStorage<ReorderableListBase, ReorderableListExposedAttribute> storage;
 
-
         private static void ConnectCallbacks(ReorderableListBase list, ReorderableListExposedAttribute attribute)
         {
             var listTarget = list.SerializedObject;
@@ -80,7 +79,6 @@ private static MethodInfo FindMethod(SerializedObject target, string methodName,
             return methodInfo;
         }
 
-
         protected override void OnGuiSafe(SerializedProperty property, GUIContent label, ReorderableListExposedAttribute attribute)
         {
             storage.ReturnItem(property, attribute).DoList(label);

Assets/Editor Toolbox/Editor/Internal/PropertyScope.cs

@@ -29,7 +29,6 @@ public PropertyScope(SerializedProperty property, GUIContent label, bool closeMa
             TryDrawLabel(rect, label);
         }
 
-
         private void HandleEvents(Rect rect)
         {
             if (property.isArray)
@@ -56,7 +55,6 @@ private void TryDrawLabel(Rect rect, GUIContent label)
             }
         }
 
-
         public void Close()
         {
             ToolboxEditorGui.CloseProperty();
@@ -73,7 +71,6 @@ public void Dispose()
             Close();
         }
 
-
         public bool IsVisible => property.isExpanded;
         public Rect LabelRect { get; private set; }
         public Rect InputRect { get; private set; }

Assets/Editor Toolbox/Editor/ToolboxDrawerModule.cs

@@ -23,7 +23,6 @@ internal static void InitializeModule()
             ToolboxEditorHandler.OnEditorReload += ReloadDrawers;
         }
 
-
         private static readonly Type decoratorDrawerBase = typeof(ToolboxDecoratorDrawer<>);
         private static readonly Type conditionDrawerBase = typeof(ToolboxConditionDrawer<>);
         private static readonly Type selfPropertyDrawerBase = typeof(ToolboxSelfPropertyDrawer<>);
@@ -261,11 +260,25 @@ internal static void UpdateDrawers(IToolboxInspectorSettings settings)
         /// </summary>
         internal static bool HasNativeTypeDrawer(Type type)
         {
-#if UNITY_2023_3_OR_NEWER
-            var parameters = new object[] { type, null, false };
-#else
-            var parameters = new object[] { type };
-#endif
+            object[] parameters;
+            var parameterInfos = getDrawerTypeForTypeMethod.GetParameters();
+            var parametersCount = parameterInfos.Length;
+            switch (parametersCount)
+            {
+                default:
+                case 1:
+                    parameters = new object[] { type };
+                    break;
+                //NOTE: Unity 2022.3.23 or above
+                case 2:
+                    parameters = new object[] { type, false };
+                    break;
+                //NOTE: Unity 2023.3.x or above
+                case 3:
+                    parameters = new object[] { type, null, false };
+                    break;
+            }
+
             var result = getDrawerTypeForTypeMethod.Invoke(null, parameters) as Type;
             return result != null && typeof(PropertyDrawer).IsAssignableFrom(result);
         }

Assets/Editor Toolbox/Editor/ToolboxEditorGui.cs

@@ -20,7 +20,6 @@ private static Rect GetLineRect(Rect rect, float thickness, float padding, bool
                 : new Rect(rect.x + padding / 2, rect.y, thickness, rect.height);
         }
 
-
         /// <summary>
         /// Draws horizontal line.
         /// Uses built-in layouting system.
@@ -248,7 +247,6 @@ public static partial class ToolboxEditorGui
     {
         private static EditorWindow lastTargetedWindow;
 
-
         /// <summary>
         /// Checks if user is still focusing the proper (searchable) window.
         /// </summary>
@@ -276,7 +274,6 @@ private static void OnPopupWindowUpdated()
             }
         }
 
-
         public static void DrawSearchablePopup(Rect rect, GUIContent label, int currentIndex, string[] options, Action<int> onSelect)
         {
             DrawSearchablePopup(rect, label, currentIndex, options, onSelect, EditorStyles.miniPullDown);

Assets/Editor Toolbox/Editor/ToolboxPropertyHandler.cs

@@ -397,7 +397,6 @@ private void ProcessLabel(SerializedProperty property, GUIContent label)
             }
         }
 
-
         /// <summary>
         /// Draw property using built-in layout system and cached <see cref="ToolboxAttributeDrawer"/>s.
         /// </summary>

Assets/Editor Toolbox/Editor/Utilities/DraggingUtility.cs

@@ -18,13 +18,11 @@ static DraggingUtility()
                 BindingFlags.NonPublic | BindingFlags.Instance);
         }
 
-
         private static readonly MethodInfo validateAssignmentMethod;
         private static readonly MethodInfo appendFoldoutValueMethod;
 
         private static readonly int dragAndDropHash = "customDragAndDrop".GetHashCode();
 
-
         public static Object ValidateAssignment(Object[] references, SerializedProperty property, Type type, bool exactType)
         {
 #if UNITY_2017_1_OR_NEWER

Assets/Editor Toolbox/Editor/Utilities/EditorGuiUtility.cs

@@ -16,7 +16,6 @@ internal static class EditorGuiUtility
 
         private static readonly Dictionary<string, Texture2D> loadedTextures = new Dictionary<string, Texture2D>();
 
-
         public static Texture2D CreateColorTexture()
         {
             return CreateColorTexture(Color.clear);
@@ -100,7 +99,6 @@ public static Texture GetHelpIcon(MessageType messageType)
             return null;
         }
 
-
         public static float FoldoutSize { get; internal set; } = 15.0f;
         public static float SpacingSize => EditorGUIUtility.standardVerticalSpacing;
         public static float HeightSize => EditorGUIUtility.singleLineHeight;