fix: Deleting a group doesn't delete children/links + batch

joezearing · Mar 22, 2021 · 01bd076 · 01bd076
1 parent 78b4395
commit 01bd076
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/src/Blazor.Diagrams.Core/Diagram.cs b/src/Blazor.Diagrams.Core/Diagram.cs
@@ -153,12 +153,13 @@ public void RemoveGroup(GroupModel group)
             if (!_groups.Remove(group))
                 return;
 
-            // Todo: batch Refresh()
-            group.Ungroup();
-            Nodes.Remove(group.Children.ToArray());
-            Links.Remove(group.AllLinks.ToArray());
-            GroupRemoved?.Invoke(group);
-            Refresh();
+            Batch(() =>
+            {
+                Nodes.Remove(group.Children.ToArray());
+                Links.Remove(group.AllLinks.ToArray());
+                group.Ungroup();
+                GroupRemoved?.Invoke(group);
+            });
         }
 
         #endregion