Skip to content

Commit

Permalink
Use property summary.
Browse files Browse the repository at this point in the history
  • Loading branch information
twastvedt authored May 7, 2024
1 parent d70525b commit c239e84
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions src/GraphMetadataViewExtension/GraphMetadataViewModel.cs
Original file line number Diff line number Diff line change
Expand Up @@ -129,14 +129,15 @@ public GraphMetadataViewModel(ViewLoadedParams viewLoadedParams, GraphMetadataVi
InitializeCommands();
}

/// <summary>
/// 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.
/// </summary>
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
Expand Down

0 comments on commit c239e84

Please sign in to comment.