check

DynamoDS · Apr 17, 2024 · 60c0df2 · 60c0df2
1 parent 4824298
commit 60c0df2
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 10 deletions.
diff --git a/src/DynamoCore/Graph/Annotations/AnnotationModel.cs b/src/DynamoCore/Graph/Annotations/AnnotationModel.cs
@@ -686,7 +686,7 @@ void SerializeCore(XmlElement element, SaveContext context)
             helper.SetAttribute("TextblockHeight", this.TextBlockHeight);
             helper.SetAttribute("backgrouund", (this.Background == null ? "" : this.Background.ToString()));
             helper.SetAttribute(nameof(IsSelected), IsSelected);
-            helper.SetAttribute(nameof(IsExpanded), IsExpanded);
+            helper.SetAttribute(nameof(IsExpanded), this.IsExpanded);
 
             //Serialize Selected models
             XmlDocument xmlDoc = element.OwnerDocument;            
@@ -766,8 +766,6 @@ protected override void DeserializeCore(XmlElement element, SaveContext context)
             RaisePropertyChanged(nameof(GroupStyleId));
             RaisePropertyChanged(nameof(AnnotationText));
             RaisePropertyChanged(nameof(Nodes));
-            // ip comment:
-            // this is called on Undo. Is VM listening?
             RaisePropertyChanged(nameof(IsExpanded));
             this.ReportPosition();
         }

diff --git a/src/DynamoCoreWpf/ViewModels/Core/AnnotationViewModel.cs b/src/DynamoCoreWpf/ViewModels/Core/AnnotationViewModel.cs
@@ -1050,12 +1050,16 @@ private void UpdateConnectorsAndPortsOnShowContents(IEnumerable<ModelBase> nodes
 
 
         /// <summary>
-        /// Handles the expansion or collapse of the annotation group in the view model.
+        /// Manages the expansion or collapse of the annotation group in the view model.
         /// </summary>
         private void ManageAnnotationMVExpansionAndCollapse()
         {
-            InPorts.Clear();
-            OutPorts.Clear();
+            if (InPorts.Any() || OutPorts.Any())
+            {
+                InPorts.Clear();
+                OutPorts.Clear();
+            }
+
             if (annotationModel.IsExpanded)
             {
                 this.ShowGroupContents();
@@ -1226,12 +1230,8 @@ private void model_PropertyChanged(object sender, System.ComponentModel.Property
                     RaisePropertyChanged(nameof(AnnotationModel.Position));
                     UpdateProxyPortsPosition();
                     break;
-
-                // ip code:
                 case nameof(IsExpanded):
-                    //UpdateAnnotationAfterUndo();
                     ManageAnnotationMVExpansionAndCollapse();
-                    //RaisePropertyChanged("IsExpanded");
                     break;
 
             }