Merge pull request #120 from arimger/develop

Develop - 0.13.2

Author: arimger

.github/workflows/test.yml

@@ -6,46 +6,39 @@ on:
 
 env:
   UNITY_LICENSE: ${{ secrets.UNITY_LICENSE }}
+  UNITY_EMAIL: ${{ secrets.UNITY_EMAIL }}
+  UNITY_PASSWORD: ${{ secrets.UNITY_PASSWORD }}
 
 jobs:
-  testRunner:
-    name: Test all modes 📝
+  testAllModes:
+    name: Test in '${{ matrix.testMode }}' 📝
     runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        testMode:
+          - playmode
+          - editmode
+          - standalone
     steps:
-      - name: Checkout code
-        uses: actions/checkout@v2
-
-      - name: Create LFS file list
-        run: git lfs ls-files -l | cut -d' ' -f1 | sort > .lfs-assets-id
-
-      - name: Restore LFS cache
-        uses: actions/cache@v2
-        id: lfs-cache
+      - uses: actions/checkout@v4
         with:
-          path: .git/lfs
-          key: ${{ runner.os }}-lfs-${{ hashFiles('.lfs-assets-id') }}
-
-      - name: Git LFS Pull
-        run: |
-          git lfs pull
-          git add .
-          git reset --hard
-
-      - name: Restore Library cache
-        uses: actions/cache@v2
+          lfs: true
+      - uses: actions/cache@v3
         with:
           path: Library
           key: Library-test-project
           restore-keys: |
             Library-test-project-
             Library-
-
-      - uses: game-ci/unity-test-runner@v2
-        id: testRunner
+      - uses: game-ci/unity-test-runner@v4
+        id: tests
         with:
-          testMode: all
-
-      - uses: actions/upload-artifact@v2
+          testMode: ${{ matrix.testMode }}
+          artifactsPath: ${{ matrix.testMode }}-artifacts
+          checkName: ${{ matrix.testMode }} Test Results
+      - uses: actions/upload-artifact@v3
+        if: always()
         with:
-          name: Test results (all modes)
-          path: ${{ steps.testRunner.outputs.artifactsPath }}
+          name: Test results for ${{ matrix.testMode }}
+          path: ${{ steps.tests.outputs.artifactsPath }}

Assets/Editor Toolbox/CHANGELOG.md

@@ -1,3 +1,14 @@
+## 0.13.2 [29.11.2024]
+
+### Added:
+- AnimationCurveSettingsAttribute
+
+### Changed:
+- Possibility to use [EditorButton] and [DynamicHelp] in nested types
+- For now SerializeReference properties without children will always be folded
+- Fix exception while building labels for generic types without arguments
+- Fix drawing SerializedDictionary if value or key types cannot be serialized
+
 ## 0.13.1 [30.08.2024]
 
 ### Changed:

Assets/Editor Toolbox/Editor/Drawers/Helpers/Extraction/ValueExtractionHelper.cs

@@ -1,6 +1,5 @@
 using System;
 using System.Collections.Generic;
-
 using UnityEditor;
 
 namespace Toolbox.Editor.Drawers
@@ -68,14 +67,8 @@ public static bool TryGetValue(string source, SerializedProperty causer, out obj
 
         public static bool TryGetValue(string source, SerializedProperty causer, out object value, out bool hasMixedValues, Func<object, object, bool> nextValuesComparer)
         {
-            var targetObjects = causer.serializedObject.targetObjects;
-            var parentObjects = new object[targetObjects.Length];
-            for (var i = 0; i < targetObjects.Length; i++)
-            {
-                var targetObject = targetObjects[i];
-                parentObjects[i] = causer.GetDeclaringObject(targetObject);
-            }
-
+            //NOTE: consider using NonAlloc implementation
+            var parentObjects = causer.GetDeclaringObjects();
             return TryGetValue(source, parentObjects, out value, out hasMixedValues, nextValuesComparer);
         }
     }

Assets/Editor Toolbox/Editor/Drawers/ISerializedPropertyContext.cs

@@ -0,0 +1,13 @@
+using System;
+using System.Reflection;
+using UnityEditor;
+
+namespace Toolbox.Editor.Drawers
+{
+    public interface ISerializedPropertyContext
+    {
+        SerializedProperty Property { get; }
+        FieldInfo FieldInfo { get; }
+        Type Type { get; }
+    }
+}

Assets/Editor Toolbox/Editor/ToolboxDrawerModule.cs.meta renamed to Assets/Editor Toolbox/Editor/Drawers/ISerializedPropertyContext.cs.meta

@@ -0,0 +1,32 @@
+using UnityEditor;
+using UnityEngine;
+
+namespace Toolbox.Editor.Drawers
+{
+    [CustomPropertyDrawer(typeof(AnimationCurveSettingsAttribute))]
+    public class AnimationCurveSettingsAttributeDrawer : PropertyDrawerBase
+    {
+        protected override void OnGUISafe(Rect position, SerializedProperty property, GUIContent label)
+        {
+            var attribute = Attribute;
+            var curveRanges = new Rect(
+                attribute.Min.x,
+                attribute.Min.y,
+                attribute.Max.x - attribute.Min.x,
+                attribute.Max.y - attribute.Min.y);
+
+            var color = attribute.Color;
+
+            EditorGUI.BeginProperty(position, label, property);
+            EditorGUI.CurveField(position, property, color, curveRanges, label);
+            EditorGUI.EndProperty();
+        }
+
+        public override bool IsPropertyValid(SerializedProperty property)
+        {
+            return base.IsPropertyValid(property) && property.propertyType == SerializedPropertyType.AnimationCurve;
+        }
+
+        private AnimationCurveSettingsAttribute Attribute => attribute as AnimationCurveSettingsAttribute;
+    }
+}

Assets/Editor Toolbox/Editor/Drawers/Regular/AnimationCurveSettingsAttributeDrawer.cs

@@ -99,16 +99,13 @@ protected override void OnGUISafe(Rect position, SerializedProperty property, GU
             }
         }
 
-
         public override bool IsPropertyValid(SerializedProperty property)
         {
             return property.propertyType == SerializedPropertyType.ObjectReference;
         }
 
-
         private AssetPreviewAttribute Attribute => attribute as AssetPreviewAttribute;
 
-
         private static class Style
         {
             internal static readonly float offset = 6.0f;

Assets/Editor Toolbox/Editor/Drawers/Regular/AnimationCurveSettingsAttributeDrawer.cs.meta

@@ -26,11 +26,10 @@ protected virtual void OnGUISafe(Rect position, SerializedProperty property, GUI
             EditorGUI.PropertyField(position, property, label);
         }
 
-
         /// <summary>
         /// Native call to return the expected height.
         /// </summary>
-        public override sealed float GetPropertyHeight(SerializedProperty property, GUIContent label)
+        public sealed override float GetPropertyHeight(SerializedProperty property, GUIContent label)
         {
             return IsPropertyValid(property)
                 ? GetPropertyHeightSafe(property, label)
@@ -40,22 +39,20 @@ public override sealed float GetPropertyHeight(SerializedProperty property, GUIC
         /// <summary>
         /// Native call to draw the provided property.
         /// </summary>
-        public override sealed void OnGUI(Rect position, SerializedProperty property, GUIContent label)
+        public sealed override void OnGUI(Rect position, SerializedProperty property, GUIContent label)
         {
             if (IsPropertyValid(property))
             {
                 OnGUISafe(position, property, label);
                 return;
             }
 
-            var warningContent = new GUIContent(property.displayName + " has invalid property drawer");
-            //create additional warning log to the console window
             ToolboxEditorLog.WrongAttributeUsageWarning(attribute, property);
             //create additional warning label based on the property name
+            var warningContent = new GUIContent(property.displayName + " has invalid property drawer");
             ToolboxEditorGui.DrawEmptyProperty(position, property, warningContent);
         }
 
-
         /// <summary>
         /// Checks if provided property can be properly handled by this drawer.
         /// </summary>

Assets/Editor Toolbox/Editor/Drawers/Regular/AssetPreviewAttributeDrawer.cs

@@ -8,7 +8,7 @@ public class DynamicHelpAttributeDrawer : ToolboxDecoratorDrawer<DynamicHelpAttr
         protected override void OnGuiBeginSafe(DynamicHelpAttribute attribute)
         {
             var sourceHandle = attribute.SourceHandle;
-            var targetObjects = ToolboxEditorHandler.CurrentTargetObjects;
+            var targetObjects = GetDeclaringObjects();
             if (ValueExtractionHelper.TryGetValue(sourceHandle, targetObjects, out var value, out var hasMixedValues))
             {
                 var messageText = hasMixedValues ? "-" : value?.ToString();