[7.x.x][Shader Graph] Fixes for 3 cases of undo not working

com.unity.shadergraph/CHANGELOG.md

@@ -9,6 +9,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 ### Fixed
 - Fixed a bug where the `Position` node would change coordinate spaces from `World` to `Absolute World` when shaders recompile. [1184617](https://issuetracker.unity3d.com/product/unity/issues/guid/1184617/)
  - Fixed a bug where `Scene Depth` nodes would stop working after adding a keyword on the blackboard. [1203333](https://issuetracker.unity3d.com/product/unity/issues/guid/1203333/)
+ - Fixed undo not being recorded properly for setting active master node, graph precision, and node defaults.
 
 ## [7.3.0] - 2020-03-11
 

com.unity.shadergraph/Editor/Data/Graphs/GraphData.cs

@@ -1059,6 +1059,9 @@ public void ReplaceWith(GraphData other)
             if (other == null)
                 throw new ArgumentException("Can only replace with another AbstractMaterialGraph", "other");
 
+            concretePrecision = other.concretePrecision;
+            m_ActiveOutputNodeGuid = other.m_ActiveOutputNodeGuid;
+
             using (var removedInputsPooledObject = ListPool<Guid>.GetDisposable())
             {
                 var removedInputGuids = removedInputsPooledObject.value;

com.unity.shadergraph/Editor/Drawing/Views/GraphEditorView.cs

@@ -156,10 +156,12 @@ public GraphEditorView(EditorWindow editorWindow, GraphData graph, MessageManage
 
                     EditorGUI.BeginChangeCheck();
                     GUILayout.Label("Precision");
-                    graph.concretePrecision = (ConcretePrecision)EditorGUILayout.EnumPopup(graph.concretePrecision, GUILayout.Width(100f));
-                    GUILayout.Space(4);
+                    var precision = (ConcretePrecision)EditorGUILayout.EnumPopup(graph.concretePrecision, GUILayout.Width(100f));
                     if (EditorGUI.EndChangeCheck())
                     {
+                        m_Graph.owner.RegisterCompleteObjectUndo("Changed Graph Precision");
+                        graph.concretePrecision = precision;
+
                         var nodeList = m_GraphView.Query<MaterialNodeView>().ToList();
                         m_ColorManager.SetNodesDirty(nodeList);
                         graph.ValidateGraph();

com.unity.shadergraph/Editor/Drawing/Views/MaterialNodeView.cs

@@ -335,6 +335,7 @@ public override void BuildContextualMenu(ContextualMenuPopulateEvent evt)
 
         void SetMasterAsActive(DropdownMenuAction action)
         {
+            node.owner.owner.RegisterCompleteObjectUndo("Change Active Master");
             node.owner.activeOutputNodeGuid = node.guid;
         }
 

com.unity.shadergraph/Editor/Drawing/Views/Slots/MultiFloatSlotControlView.cs

@@ -70,11 +70,12 @@ void AddField(Vector4 initialValue, int index, string subLabel)
                         m_Node.Dirty(ModificationScope.Node);
                     }
                 });
-            // Reset UndoGroup when done editing input field
+            // Reset UndoGroup when done editing input field & update title
             field.Q("unity-text-input").RegisterCallback<FocusOutEvent>(evt =>
-                {
-                    m_UndoGroup = -1;
-                });
+            {
+                m_Node.owner.owner.isDirty = true;
+                m_UndoGroup = -1;
+            });
             Add(field);
         }
     }