RunAll disabled in CustomNodes

DynamoDS · Nov 15, 2024 · e96fd0d · e96fd0d
1 parent 5d3052c
commit e96fd0d
Show file tree

Hide file tree

Showing 5 changed files with 55 additions and 2 deletions.
diff --git a/TuneUp/Properties/Resources.Designer.cs b/TuneUp/Properties/Resources.Designer.cs
diff --git a/TuneUp/Properties/Resources.en-US.resx b/TuneUp/Properties/Resources.en-US.resx
@@ -214,6 +214,9 @@
   <data name="ToolTip_RunAll" xml:space="preserve">
     <value>Execute entire graph, including nodes with no change since the latest run.</value>
   </data>
+  <data name="ToolTip_RunAllDisabled" xml:space="preserve">
+    <value>Custom Node mode does not support Run All.</value>
+  </data>
   <data name="ToolTip_TotalExecutionTime" xml:space="preserve">
     <value>Combined execution time of latest and previous run.</value>
   </data>

diff --git a/TuneUp/Properties/Resources.resx b/TuneUp/Properties/Resources.resx
@@ -214,6 +214,9 @@
   <data name="ToolTip_RunAll" xml:space="preserve">
     <value>Execute entire graph, including nodes with no change since the latest run.</value>
   </data>
+  <data name="ToolTip_RunAllDisabled" xml:space="preserve">
+    <value>Custom Node mode does not support Run All.</value>
+  </data>
   <data name="ToolTip_TotalExecutionTime" xml:space="preserve">
     <value>Combined execution time of latest and previous run.</value>
   </data>

diff --git a/TuneUp/TuneUpWindow.xaml b/TuneUp/TuneUpWindow.xaml
@@ -345,6 +345,9 @@
                             </BeginStoryboard>
                         </Trigger.EnterActions>
                     </Trigger>
+                    <Trigger Property="IsEnabled" Value="False">
+                        <Setter Property="Foreground" Value="{StaticResource DisabledForegroundBrush}" />
+                    </Trigger>
                 </Style.Triggers>
             </Style>
             <!--  Context Menu style for Export button *  -->
@@ -483,10 +486,11 @@
             Content="{x:Static resx:Resources.Button_RunAll}"
             Cursor="Hand"
             IsEnabled="{Binding Path=IsRecomputeEnabled}"
-            Style="{StaticResource ButtonStyleTuneUp}" >
+            Style="{StaticResource ButtonStyleTuneUp}"
+            ToolTipService.ShowOnDisabled="True" >
             <Button.ToolTip>
                 <ToolTip Style="{StaticResource ToolTipPointerTopRight}">
-                    <TextBlock Text="{x:Static resx:Resources.ToolTip_RunAll}" Style="{StaticResource ToolTipContent}"/>
+                    <TextBlock Style="{StaticResource ToolTipContent}" Text="{Binding RunAllTooltipMessage}" />
                 </ToolTip>
             </Button.ToolTip>
         </Button>

diff --git a/TuneUp/TuneUpWindowViewModel.cs b/TuneUp/TuneUpWindowViewModel.cs
@@ -119,6 +119,7 @@ private set
                 {
                     isRecomputeEnabled = value;
                     RaisePropertyChanged(nameof(IsRecomputeEnabled));
+                    RaisePropertyChanged(nameof(RunAllTooltipMessage));
                 }
             }
         }
@@ -279,6 +280,7 @@ public string SortingOrder
         public const string SortByName = "name";
         public const string SortByNumber = "number";
         public const string SortByTime = "time";
+        public string RunAllTooltipMessage => IsRecomputeEnabled ? Resources.ToolTip_RunAll : Resources.ToolTip_RunAllDisabled;
 
         #endregion
 
@@ -998,6 +1000,17 @@ private void OnCurrentWorkspaceChanged(IWorkspaceModel workspace)
         {
             // Profiling needs to be enabled per workspace so mark it false after switching
             isProfilingEnabled = false;
+
+            // Disable IsRecomputeEnabled if the workspace is a CustomNodeWorkspaceModel
+            IsRecomputeEnabled = !(workspace is CustomNodeWorkspaceModel);
+
+            if (workspace is CustomNodeWorkspaceModel)
+            {
+                ResetForCustomNodeWorkspace();
+                CurrentWorkspace = null;
+                return;
+            }
+
             CurrentWorkspace = workspace as HomeWorkspaceModel;
         }
 
@@ -1376,6 +1389,27 @@ private void RefreshGroupNodeUI()
             RaisePropertyChanged(nameof(ProfiledNodesCollectionPreviousRun));
         }
 
+        /// <summary>
+        /// Resets the UI and profiling state when the workspace is a custom node.
+        /// </summary>
+        private void ResetForCustomNodeWorkspace()
+        {
+            // Clear node data for custom nodes
+            ProfiledNodesLatestRun.Clear();
+            ProfiledNodesPreviousRun.Clear();
+            ProfiledNodesNotExecuted.Clear();
+
+            // Trigger UI refresh
+            RaisePropertyChanged(nameof(ProfiledNodesCollectionLatestRun));
+            RaisePropertyChanged(nameof(ProfiledNodesCollectionPreviousRun));
+            RaisePropertyChanged(nameof(ProfiledNodesCollectionNotExecuted));
+
+            // Reset execution time stats
+            LatestGraphExecutionTime = PreviousGraphExecutionTime = TotalGraphExecutionTime = defaultExecutionTime;
+
+            UpdateTableVisibility();
+        }
+
         #endregion
 
         #region Dispose or setup