From 011915ef42b0d4093a14ac89f0859356034f2560 Mon Sep 17 00:00:00 2001 From: Spike Curtis Date: Thu, 22 May 2025 14:38:13 +0400 Subject: [PATCH] fix: changes dummy workspace logic to include starting and stopping --- App/ViewModels/TrayWindowViewModel.cs | 19 ++++++++++++++++++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git a/App/ViewModels/TrayWindowViewModel.cs b/App/ViewModels/TrayWindowViewModel.cs index cfa5163..4d493c9 100644 --- a/App/ViewModels/TrayWindowViewModel.cs +++ b/App/ViewModels/TrayWindowViewModel.cs @@ -207,7 +207,7 @@ private void UpdateFromRpcModel(RpcModel rpcModel) // For every stopped workspace that doesn't have any agents, add a // dummy agent row. foreach (var workspace in rpcModel.Workspaces.Where(w => - w.Status == Workspace.Types.Status.Stopped && !workspacesWithAgents.Contains(w.Id))) + ShouldShowDummy(w) && !workspacesWithAgents.Contains(w.Id))) { if (!Uuid.TryFrom(workspace.Id.Span, out var uuid)) continue; @@ -372,4 +372,21 @@ public void Exit() { _ = ((App)Application.Current).ExitApplication(); } + + private static bool ShouldShowDummy(Workspace workspace) + { + switch (workspace.Status) + { + case Workspace.Types.Status.Unknown: + case Workspace.Types.Status.Pending: + case Workspace.Types.Status.Starting: + case Workspace.Types.Status.Stopping: + case Workspace.Types.Status.Stopped: + return true; + // TODO: should we include and show a different color than Gray for workspaces that are + // failed, canceled or deleting? + default: + return false; + } + } }