feat: update version code & update docs & sorting code

Author: SolarianZ

Editor/Scripts/GraphView/PlayableGraphView.cs

@@ -36,13 +36,13 @@ public void Update(PlayableGraph playableGraph)
                 ClearView();
             }
 
-            PopulateView();
+            UpdateView();
 
             CalculateLayout();
 
             if (needFrameAll)
             {
-                // wait for view initialize at least 2 frames
+                // wait at least 2 frames for view initialization
                 EditorApplication.delayCall += () =>
                 {
                     EditorApplication.delayCall += () => { FrameAll(); };
@@ -61,7 +61,7 @@ private void ClearView()
             _rootOutputNodes.Clear();
         }
 
-        private void PopulateView()
+        private void UpdateView()
         {
             if (!_playableGraph.IsValid())
             {
@@ -93,6 +93,7 @@ private void PopulateView()
                 _rootOutputNodes.Add(playableOutputNode);
             }
 
+            // check and update nodes
             for (int i = _rootOutputNodes.Count - 1; i >= 0; i--)
             {
                 var rootOutputNode = _rootOutputNodes[i];
@@ -131,7 +132,7 @@ private void CalculateLayout()
 
                 outputNode.CalculateLayout(origin);
 
-                origin.y += treeSize.y + NodeLayoutInfo.VerticalSpace;
+                origin.y += treeSize.y + GraphViewNode.VerticalSpace;
             }
         }
     }

Editor/Scripts/Node/GraphViewNode.cs

@@ -12,7 +12,7 @@ public enum NodeFlag : uint
     {
         None = 0,
         Active = 1 << 0,
-        Dirty = 1 << 1,
+        //Dirty = 1 << 1,
     }
 
     public readonly struct NodeInput
@@ -49,15 +49,14 @@ public abstract class GraphViewNode : UNode
 
         protected UGraphView Container { get; set; }
 
-        public IReadOnlyList<NodeInput> Inputs => InternalInputs;
-
         protected List<NodeInput> InternalInputs { get; } = new List<NodeInput>();
 
-        protected GraphViewNode Parent { get; private set; }
+        //protected GraphViewNode Parent { get; private set; }
 
 
         public virtual void Update() { }
 
+
         #region Description
 
         private StringBuilder _descBuilder;
@@ -79,19 +78,18 @@ public string GetStateDescription()
 
         protected abstract void AppendStateDescriptions(StringBuilder descBuilder);
 
-
         #endregion
 
 
         #region Hierarchy
 
-        public virtual void AddToContainer(UGraphView container)
+        public void AddToContainer(UGraphView container)
         {
             Container = container;
             Container.AddElement(this);
         }
 
-        public virtual void RemoveFromContainer()
+        public void RemoveFromContainer()
         {
             // self
             Container.RemoveElement(this);
@@ -120,12 +118,18 @@ protected Port InstantiatePort<TPort>(Direction direction)
 
         #region Layout
 
+        public const int HorizontalSpace = 80;
+
+        public const int VerticalSpace = 80;
+
+        public static readonly Vector2 StandardNodeSize = new Vector2(400, 150);
+
         private Vector2? _hierarchySize;
 
 
         public Vector2 GetNodeSize()
         {
-            return NodeLayoutInfo.StandardNodeSize;
+            return StandardNodeSize;
             //return worldBound.size;
         }
 
@@ -136,22 +140,22 @@ public Vector2 GetHierarchySize()
                 return _hierarchySize.Value;
             }
 
-            if (Inputs.Count == 0)
+            if (InternalInputs.Count == 0)
             {
                 _hierarchySize = GetNodeSize();
                 return _hierarchySize.Value;
             }
 
             var subHierarchySize = Vector2.zero;
-            for (int i = 0; i < Inputs.Count; i++)
+            for (int i = 0; i < InternalInputs.Count; i++)
             {
-                var childSize = Inputs[i].Node.GetHierarchySize();
+                var childSize = InternalInputs[i].Node.GetHierarchySize();
                 subHierarchySize.x = Mathf.Max(subHierarchySize.x, childSize.x);
                 subHierarchySize.y += childSize.y;
             }
-            subHierarchySize.y += (Inputs.Count - 1) * NodeLayoutInfo.VerticalSpace;
+            subHierarchySize.y += (InternalInputs.Count - 1) * VerticalSpace;
 
-            var hierarchySize = GetNodeSize() + new Vector2(NodeLayoutInfo.HorizontalSpace, 0);
+            var hierarchySize = GetNodeSize() + new Vector2(HorizontalSpace, 0);
             hierarchySize.y = Mathf.Max(hierarchySize.y, subHierarchySize.y);
             _hierarchySize = hierarchySize;
 
@@ -164,10 +168,10 @@ public void CalculateLayout(Vector2 origin)
             var nodePos = CalculateSubTreeRootNodePosition(subTreeSize, origin);
             SetPosition(new Rect(nodePos, Vector2.zero));
 
-            origin.x -= GetNodeSize().x - NodeLayoutInfo.HorizontalSpace;
-            for (int i = 0; i < Inputs.Count; i++)
+            origin.x -= GetNodeSize().x - HorizontalSpace;
+            for (int i = 0; i < InternalInputs.Count; i++)
             {
-                var childNode = Inputs[i];
+                var childNode = InternalInputs[i];
                 var childHierarchySize = childNode.Node.GetHierarchySize();
                 childNode.Node.CalculateLayout(origin);
 

Editor/Scripts/Node/NodeLayoutInfo.cs

@@ -72,7 +72,7 @@ public override void Update()
                 InternalInputs.Add(new NodeInput(edge, inputPlayableNode, i));
             }
 
-            // update child nodes
+            // check and update children
             for (int i = InternalInputs.Count - 1; i >= 0; i--)
             {
                 var input = InternalInputs[i];

Editor/Scripts/Node/PlayableNode.cs

@@ -58,6 +58,7 @@ public override void Update()
                 }
             }
 
+            // check and update children
             for (int i = InternalInputs.Count - 1; i >= 0; i--)
             {
                 var input = InternalInputs[i];

Editor/Scripts/Node/PlayableOutputNode.cs

@@ -38,7 +38,6 @@ public static string DurationToString(this Playable playable, string format = "F
             return durationStr;
         }
 
-
         public static void SetNodeStyle(this GraphViewNode node, Color nodeColor,
             float titleFontSize = 15, Color? titleColor = null)
         {
@@ -51,10 +50,22 @@ public static void SetNodeStyle(this GraphViewNode node, Color nodeColor,
         }
 
 
+        #region Port Color
+
         // Ensure edge is always visible.
         public const float ColorAlphaFactor = 1f / 9;
 
 
+        public static Color GetPortColor(float weight)
+        {
+            var alpha = (weight + ColorAlphaFactor) / (1 + ColorAlphaFactor);
+            return new Color(1, 1, 1, alpha);
+        }
+
+        #endregion
+
+
+
         public static Color GetButtonBackgroundColor(bool isChecked)
         {
             if (isChecked)
@@ -77,24 +88,22 @@ public static Color GetNodeInspectorBackgroundColor()
                 new Color32(175, 175, 175, 255); // light
         }
 
+
+        #region Node Color
+
         public static Color GetNodeInspectorTextColor()
         {
             return EditorGUIUtility.isProSkin ?
                 new Color32(255, 255, 255, 255) : // dark
                 new Color32(0, 0, 0, 255); // light
         }
 
-        public static Color GetPortColor(float weight)
-        {
-            var alpha = (weight + ColorAlphaFactor) / (1 + ColorAlphaFactor);
-            return new Color(1, 1, 1, alpha);
-        }
-
         public static Color GetNodeInvalidColor()
         {
             return Color.red;
         }
 
+        // (0, ?, 255, 255) for PlayableOutput nodes
         public static Color GetPlayableOutputNodeColor(this ref PlayableOutput playableOutput)
         {
             if (playableOutput.IsPlayableOutputOfType<AnimationPlayableOutput>())
@@ -111,40 +120,37 @@ public static Color GetPlayableOutputNodeColor(this ref PlayableOutput playableO
             return GetRandomColorForType(playableOutput.GetPlayableOutputType());
         }
 
+        // (0, 255, ?, 255) for Playable nodes
         public static Color GetPlayableNodeColor(this ref Playable playable)
         {
             if (playable.IsPlayableOfType<AnimationClipPlayable>())
             {
-                return new Color32(0, 255, 102, 255);
+                return new Color32(0, 255, 51, 255);
             }
 
             if (playable.IsPlayableOfType<AnimationMixerPlayable>())
             {
-                return new Color32(0, 255, 153, 255);
+                return new Color32(0, 255, 102, 255);
             }
 
             if (playable.IsPlayableOfType<AnimationLayerMixerPlayable>())
             {
-                return new Color32(0, 255, 204, 255);
+                return new Color32(0, 255, 153, 255);
             }
 
-            return GetRandomColorForType(playable.GetPlayableType());
-        }
-
-        public static Color GetRandomColorForType(this Type type)
-        {
-            if (_colorCache.TryGetValue(type, out var color))
+            if (playable.IsPlayableOfType<AnimationScriptPlayable>())
             {
-                return color;
+                return new Color32(0, 255, 204, 255);
             }
 
-            color = ColorPool[URandom.Range(0, ColorPool.Count)];
-            _colorCache[type] = color;
-
-            return color;
+            return GetRandomColorForType(playable.GetPlayableType());
         }
 
+        #endregion
+
 
+        // reserve (0, ?, 255, 255) for PlayableOutput nodes
+        // reserve (0, 255, ?, 255) for Playable nodes
         public static readonly IReadOnlyList<Color32> ColorPool = new Color32[]
         {
             new Color32(255,0,255,255), new Color32(255,0,153,255),
@@ -159,6 +165,19 @@ public static Color GetRandomColorForType(this Type type)
         private static readonly Dictionary<Type, Color32> _colorCache = new Dictionary<Type, Color32>();
 
 
+        public static Color GetRandomColorForType(this Type type)
+        {
+            if (_colorCache.TryGetValue(type, out var color))
+            {
+                return color;
+            }
+
+            color = ColorPool[URandom.Range(0, ColorPool.Count)];
+            _colorCache[type] = color;
+
+            return color;
+        }
+
         public static void ClearGlobalCache()
         {
             _colorCache.Clear();