- Added more tests

- Renamed TagEffect to TagComponent
- Renamedd TransitionEffect to TransitionComponent
- Added isActive boolean to MotionComponent
- Improved MotionComponentRegistry
 - Cleaned up comments
 - Simplified syntax
- Improved TextMotionProPlugin
 - Refactored some names to improve readability
 - Added helper method to prevent direct access to component registry
 - Added helper method to find ComponentDescriptor based on type
- Mergerd transition and tag contexts together
- Moved Standard components to /Runtime/Components/Standard
- Removed redundant folders

Author: BluePixelDev

.github/workflows/test.yaml

@@ -14,14 +14,6 @@ jobs:
       with:
         lfs: true
 
-    #Cache
-    - uses: actions/cache@v4
-      with:
-          path: Library
-          key: Library-${{ hashFiles('Assets/**', 'Packages/**', 'ProjectSettings/**') }}
-          restore-keys: |
-            Library-
-    
     # Move the package to a subdirectory
     - name: Move package to subdirectory
       run: |

Editor/MotionCollection/ElementContainer.uxml

@@ -1,8 +1,11 @@
 <engine:UXML xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:engine="UnityEngine.UIElements" xmlns:editor="UnityEditor.UIElements" noNamespaceSchemaLocation="../../../UIElementsSchema/UIElements.xsd" editor-extension-mode="True">
     <engine:VisualElement name="element" style="margin-bottom: 0;">
-        <editor:Toolbar name="header" style="border-top-width: 1px; border-bottom-width: 0; background-color: rgba(255, 255, 255, 0.05); align-items: center; justify-content: space-between; margin-top: 0; -unity-font-style: bold;">
-            <engine:Label text="Wave Effect" name="label" style="justify-content: flex-start; align-self: auto; -unity-text-align: middle-left;" />
-            <editor:ToolbarButton text="⋮" name="options-button" style="font-size: 23px; border-right-width: 0; border-left-width: 0; background-color: rgba(60, 60, 60, 0);" />
+        <editor:Toolbar name="header" style="border-top-width: 1px; border-bottom-width: 0; background-color: rgba(255, 255, 255, 0.05); align-items: center; justify-content: flex-start; margin-top: 0; -unity-font-style: bold;">
+            <engine:Toggle name="active-toggle" binding-path="isActive" value="false" style="margin-top: 0; margin-right: 0; margin-bottom: 0; margin-left: 0; padding-right: 4px; padding-left: 4px;" />
+            <engine:VisualElement name="foldout-box" style="flex-grow: 1; flex-direction: row;">
+                <engine:Label text="Wave Effect" name="label" style="justify-content: flex-start; align-self: auto; -unity-text-align: middle-left;" />
+                <editor:ToolbarButton text="⋮" name="options-button" style="font-size: 23px; border-right-width: 0; border-left-width: 0; background-color: rgba(60, 60, 60, 0); margin-left: auto;" />
+            </engine:VisualElement>
         </editor:Toolbar>
         <engine:GroupBox name="body" style="margin-top: 0; display: none;" />
     </engine:VisualElement>

Editor/MotionCollection/MotionCollectionEditor.cs

@@ -0,0 +1,135 @@
+using BP.TextMotion;
+using System.Reflection;
+using UnityEditor;
+using UnityEditor.UIElements;
+using UnityEngine;
+using UnityEngine.UIElements;
+
+namespace BP.TextMotionEditor
+{
+    [CustomPropertyDrawer(typeof(MotionComponentRegistry<>), true)]
+    public class MotionComponentRegistryEditor : PropertyDrawer
+    {
+        [SerializeField] private VisualTreeAsset listContainerAsset;
+        [SerializeField] private VisualTreeAsset elementContainerAsset;
+
+        private VisualElement listRoot;
+        private SerializedProperty listProperty;
+
+        public override VisualElement CreatePropertyGUI(SerializedProperty property)
+        {
+            var root = listContainerAsset.CloneTree();
+            listRoot = root.Q<VisualElement>("elements-list");
+            listProperty = property.FindPropertyRelative("components");
+
+            RefreshList();
+            return root;
+        }
+
+        private void RefreshList()
+        {
+            listRoot.Clear();
+            for (int i = 0; i < listProperty.arraySize; i++)
+            {
+                listRoot.Add(CreateComponentElement(i));
+            }
+        }
+
+        private VisualElement CreateComponentElement(int index)
+        {
+            var elementProp = listProperty.GetArrayElementAtIndex(index);
+            var motionComponent = elementProp.objectReferenceValue as MotionComponent;
+            if (motionComponent == null) return new Label("Missing component");
+
+            var container = elementContainerAsset.CloneTree();
+            var header = container.Q<VisualElement>("header");
+            var body = container.Q<VisualElement>("body");
+            var optionsButton = container.Q<ToolbarButton>("options-button");
+
+            SetupHeader(header, motionComponent, body);
+            SetupOptionsMenu(optionsButton, index);
+
+            return container;
+        }
+
+        private void SetupHeader(VisualElement header, MotionComponent component, VisualElement body)
+        {
+            var type = component.GetType();
+            var attr = type.GetCustomAttribute<TextMotionAttribute>();
+            header.Q<Label>("label").text = attr?.DisplayName ?? type.Name;
+
+            var foldoutClickable = header.Q("foldout-box");
+            var activeToggle = header.Q<Toggle>("active-toggle");
+
+            var so = new SerializedObject(component);
+            var isFoldedProp = so.FindProperty("isFolded");
+            var isActiveProp = so.FindProperty("isActive");
+
+            activeToggle.BindProperty(isActiveProp);
+
+            body.Add(CreateEditorElement(so));
+            UpdateBodyVisibility(body, isFoldedProp.boolValue);
+
+            foldoutClickable.RegisterCallback<ClickEvent>(_ =>
+            {
+                isFoldedProp.boolValue = !isFoldedProp.boolValue;
+                so.ApplyModifiedProperties();
+                UpdateBodyVisibility(body, isFoldedProp.boolValue);
+            });
+
+            foldoutClickable.TrackPropertyValue(isFoldedProp, prop =>
+            {
+                UpdateBodyVisibility(body, prop.boolValue);
+            });
+        }
+
+        private void UpdateBodyVisibility(VisualElement body, bool isFolded)
+        {
+            body.style.display = isFolded ? DisplayStyle.Flex : DisplayStyle.None;
+        }
+
+        private void SetupOptionsMenu(ToolbarButton button, int index)
+        {
+            button.clicked += () =>
+            {
+                var menu = new GenericMenu();
+
+                // Remove option
+                menu.AddItem(new GUIContent("Remove"), false, () =>
+                {
+                    RemoveComponentAt(index);
+                });
+                menu.DropDown(button.worldBound);
+            };
+        }
+
+        private void RemoveComponentAt(int index)
+        {
+            var targetObj = listProperty.serializedObject.targetObject;
+            Undo.RecordObject(targetObj, "Remove Motion Component");
+
+            listProperty.DeleteArrayElementAtIndex(index);
+            listProperty.serializedObject.ApplyModifiedProperties();
+
+            RefreshList();
+        }
+
+        private VisualElement CreateEditorElement(SerializedObject serializedObject)
+        {
+            var editor = Editor.CreateEditor(serializedObject.targetObject);
+            if (editor == null)
+                return new Label("No custom editor available");
+
+            var inspectorGUI = editor.CreateInspectorGUI();
+            if (inspectorGUI != null)
+                return inspectorGUI;
+
+            return new IMGUIContainer(() =>
+            {
+                serializedObject.Update();
+                editor.OnInspectorGUI();
+                serializedObject.ApplyModifiedProperties();
+            });
+        }
+    }
+}

Editor/MotionCollection/MotionComponentRegistryEditor.cs

@@ -7,15 +7,15 @@
 
 namespace BP.TextMotionEditor
 {
-    public readonly struct MotionComponentDescriptor
+    public readonly struct ComponentDescriptor
     {
         public readonly Type Type;
         public readonly string DisplayName;
         public readonly string Description;
         public readonly string Category;
         public readonly TextMotionRole Role;
 
-        public MotionComponentDescriptor(Type type, string displayName, TextMotionRole role, string description, string category = null)
+        public ComponentDescriptor(Type type, string displayName, TextMotionRole role, string description, string category = null)
         {
             Type = type;
             DisplayName = displayName;
@@ -26,17 +26,18 @@ public MotionComponentDescriptor(Type type, string displayName, TextMotionRole r
     }
 
     [InitializeOnLoad]
-    internal static class MotionComponentRegistry
+    internal static class TextMotionProPlugin
     {
-        public static readonly List<MotionComponentDescriptor> Components = new();
-        static MotionComponentRegistry() => Init();
+        private static readonly List<ComponentDescriptor> Components = new();
+        static TextMotionProPlugin() => Init();
 
         public static void Init()
         {
             Components.Clear();
-            DiscoverComponents<TagEffect, TextMotionAttribute>();
+            DiscoverComponents<TagComponent, TextMotionAttribute>();
         }
 
+        // Discovery process
         private static void DiscoverComponents<TBase, TAttr>() where TBase : MotionComponent where TAttr : Attribute
         {
             foreach (var type in GetTypesWithAttribute<TAttr>())
@@ -47,12 +48,11 @@ private static void DiscoverComponents<TBase, TAttr>() where TBase : MotionCompo
 
                 string name = (string)typeof(TAttr).GetProperty("DisplayName")?.GetValue(attr) ?? type.Name;
                 string desc = (string)typeof(TAttr).GetProperty("Description")?.GetValue(attr);
-                string category = (string)typeof(TAttr).GetProperty("Category").GetValue(attr);
+                string category = (string)typeof(TAttr).GetProperty("Category")?.GetValue(attr);
                 TextMotionRole role = (TextMotionRole)typeof(TAttr).GetProperty("Role").GetValue(attr);
-                Components.Add(new MotionComponentDescriptor(type, name, role, desc, category));
+                Components.Add(new ComponentDescriptor(type, name, role, desc, category));
             }
         }
-
         private static IEnumerable<Type> GetTypesWithAttribute<T>() where T : Attribute
         {
             return AppDomain.CurrentDomain.GetAssemblies()
@@ -64,5 +64,22 @@ private static IEnumerable<Type> GetTypesWithAttribute<T>() where T : Attribute
                })
                .Where(t => t.GetCustomAttribute<T>() != null);
         }
+
+        // Utilities
+        public static bool TryGetComponentOfType(Type type, out ComponentDescriptor result)
+        {
+            foreach (var desc in Components)
+            {
+                if (desc.Type == type)
+                {
+                    result = desc;
+                    return true;
+                }
+            }
+
+            result = default;
+            return false;
+        }
+        public static ComponentDescriptor[] GetComponents() => Components.ToArray();
     }
 }

Editor/MotionCollection/MotionCollectionEditor.cs.meta renamed to Editor/MotionCollection/MotionComponentRegistryEditor.cs.meta

@@ -0,0 +1,15 @@
+using UnityEngine;
+
+namespace BP.TextMotion
+{
+    public abstract class MotionComponent : ScriptableObject
+    {
+        [SerializeField, HideInInspector] private bool isFolded = false;
+        [SerializeField, HideInInspector] protected bool isActive = false;
+
+        public abstract string Key { get; }
+        public virtual bool IsActive() => isActive;
+        public virtual void ResetContext(TextMotionPro renderer) { }
+        public virtual void Release() { }
+    }
+}