Fix refresh bugs for the undo history form.

Author: SuuperW

src/BizHawk.Client.Common/movie/tasproj/TasMovie.Editing.cs

@@ -164,15 +164,15 @@ public void RemoveFrames(int removeStart, int removeUpTo)
 			ShiftBindedMarkers(removeUpTo, removeStart - removeUpTo);
 
 			Changes = true;
-			InvalidateAfter(removeStart);
-
 			ChangeLog.AddRemoveFrames(
 				removeStart,
 				removeUpTo,
 				removedInputs.ToList(),
 				removedMarkers,
 				$"Remove frames {removeStart}-{removeUpTo - 1}"
 			);
+
+			InvalidateAfter(removeStart);
 		}
 
 		public void InsertInput(int frame, string inputState)
@@ -188,9 +188,9 @@ public void InsertInput(int frame, IEnumerable<string> inputLog)
 			ShiftBindedMarkers(frame, inputLog.Count());
 
 			Changes = true;
-			InvalidateAfter(frame);
-
 			ChangeLog.AddInsertInput(frame, inputLog.ToList(), $"Insert {inputLog.Count()} frame(s) at {frame}");
+
+			InvalidateAfter(frame);
 		}
 
 		public void InsertInput(int frame, IEnumerable<IController> inputStates)
@@ -258,9 +258,9 @@ public void InsertEmptyFrame(int frame, int count = 1)
 			ShiftBindedMarkers(frame, count);
 
 			Changes = true;
-			InvalidateAfter(frame);
-
 			ChangeLog.AddInsertFrames(frame, count, $"Insert {count} empty frame(s) at {frame}");
+
+			InvalidateAfter(frame);
 		}
 
 		private void ExtendMovieForEdit(int numFrames)
@@ -295,12 +295,12 @@ public void ToggleBoolState(int frame, string buttonName)
 
 			var adapter = GetInputState(frame);
 			adapter.SetBool(buttonName, !adapter.IsPressed(buttonName));
-
 			Log[frame] = Bk2LogEntryGenerator.GenerateLogEntry(adapter);
-			Changes = true;
-			InvalidateAfter(frame);
 
+			Changes = true;
 			ChangeLog.AddBoolToggle(frame, buttonName, !adapter.IsPressed(buttonName), $"Toggle {buttonName}: {frame}");
+
+			InvalidateAfter(frame);
 		}
 
 		public void SetBoolState(int frame, string buttonName, bool val)
@@ -318,9 +318,9 @@ public void SetBoolState(int frame, string buttonName, bool val)
 
 			if (old != val)
 			{
-				InvalidateAfter(frame);
 				Changes = true;
 				ChangeLog.AddBoolToggle(frame, buttonName, old, $"Set {buttonName}({(val ? "On" : "Off")}): {frame}");
+				InvalidateAfter(frame);
 			}
 		}
 
@@ -372,9 +372,9 @@ public void SetAxisState(int frame, string buttonName, int val)
 
 			if (old != val)
 			{
-				InvalidateAfter(frame);
 				Changes = true;
 				ChangeLog.AddAxisChange(frame, buttonName, old, val, $"Set {buttonName}({val}): {frame}");
+				InvalidateAfter(frame);
 			}
 		}
 

src/BizHawk.Client.EmuHawk/tools/TAStudio/TAStudio.ListView.cs

@@ -874,11 +874,18 @@ public bool FrameEdited(int frame)
 					RefreshDialog();
 					return true;
 				}
-				else if (TasView.RowCount != CurrentTasMovie.InputLogLength + 1)
+				else
 				{
-					// Row count must always be kept up to date even if last row is not directly visible.
-					TasView.RowCount = CurrentTasMovie.InputLogLength + 1;
-					return true;
+					if (_undoForm != null && !_undoForm.IsDisposed)
+					{
+						_undoForm.UpdateValues();
+					}
+					if (TasView.RowCount != CurrentTasMovie.InputLogLength + 1)
+					{
+						// Row count must always be kept up to date even if last row is not directly visible.
+						TasView.RowCount = CurrentTasMovie.InputLogLength + 1;
+						return true;
+					}
 				}
 			}