From c239e8466900b22aaf49202232ebc3fd61aea47c Mon Sep 17 00:00:00 2001 From: Trygve Wastvedt Date: Tue, 7 May 2024 13:00:37 -0500 Subject: [PATCH] Use property summary. --- .../GraphMetadataViewModel.cs | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/GraphMetadataViewExtension/GraphMetadataViewModel.cs b/src/GraphMetadataViewExtension/GraphMetadataViewModel.cs index 69ea175b2bd..6af7e69ba64 100644 --- a/src/GraphMetadataViewExtension/GraphMetadataViewModel.cs +++ b/src/GraphMetadataViewExtension/GraphMetadataViewModel.cs @@ -129,14 +129,15 @@ public GraphMetadataViewModel(ViewLoadedParams viewLoadedParams, GraphMetadataVi InitializeCommands(); } + /// + /// This event is triggered when a new workspace is opened or when the current workspace is cleared. + /// This event manages state of the workspace properties (ie GraphDescription, GraphAuthor, HelpLink, Thumbnail) + /// as well as the custom properties in the extension which do not live in the workspace model. + /// private void OnCurrentWorkspaceChanged(Graph.Workspaces.IWorkspaceModel obj) { - //This event is triggered when a new workspace is opened or when the current workspace is cleared. - //This event manages state of the workspace properties (ie GraphDescription, GraphAuthor, HelpLink, Thumbnail) - //as well as the custom properties in the extension which do not live in the workspace model. - //Todo review if the workspace properties should be managed in the Workspace model. - //Do to the fact that Dynamo oftens leaves the workspace objs in memory and resets their properties when you open a new workspace + //Due to the fact that Dynamo often leaves the workspace objs in memory and resets their properties when you open a new workspace, //the management of state is not straightforward. However it may make more sense to update those properteis with the clearing logic. //Handle the case of a custom workspace model opening