From 7ae27fa0440800d69df4ff7c2871361ee9e4bb8a Mon Sep 17 00:00:00 2001
From: rcourtman <courtmanr@gmail.com>
Date: Sat, 16 May 2026 21:16:36 +0100
Subject: [PATCH] proxmox(overview): drop duplicate status column, flatten
 guest list

Two duplication issues on /proxmox/overview after the v5-style hosts table
landed:

1. The nodes table rendered the status both as a colored dot beside the
   node name and as a separate Status badge column. The dot already
   conveys it; drop the Status column.
2. The embedded WorkloadsSurface was forced into `grouped` mode, which
   re-rendered NodeGroupHeader rows for the same hosts the new nodes
   table now shows on top. Switch the forced grouping to `flat`; each
   guest row keeps its Node column so the per-row attribution is intact
   without the duplicated headers.
---
 .../src/features/proxmox/ProxmoxNodesTable.tsx       | 12 +-----------
 .../src/features/proxmox/ProxmoxPageSurface.tsx      |  2 +-
 2 files changed, 2 insertions(+), 12 deletions(-)

diff --git a/frontend-modern/src/features/proxmox/ProxmoxNodesTable.tsx b/frontend-modern/src/features/proxmox/ProxmoxNodesTable.tsx
index ee803f5ff..462cdf23d 100644
--- a/frontend-modern/src/features/proxmox/ProxmoxNodesTable.tsx
+++ b/frontend-modern/src/features/proxmox/ProxmoxNodesTable.tsx
@@ -16,7 +16,7 @@ import {
   TableHeader,
   TableRow,
 } from '@/components/shared/Table';
-import { getSimpleStatusIndicator, getStatusIndicatorBadgeToneClasses } from '@/utils/status';
+import { getSimpleStatusIndicator } from '@/utils/status';
 import { asTrimmedString } from '@/utils/stringUtils';
 import { normalizeDiskArray } from '@/utils/format';
 import { buildMetricKeyForUnifiedResource } from '@/utils/metricsKeys';
@@ -138,7 +138,6 @@ export const ProxmoxNodesTable: Component<{
               <TableHeader class="bg-surface-alt text-muted border-b border-border">
                 <TableRow class="text-left text-[10px] uppercase tracking-wide">
                   <TableHead class="px-3 py-2 font-medium">Node</TableHead>
-                  <TableHead class="px-3 py-2 font-medium">Status</TableHead>
                   <TableHead class="px-3 py-2 font-medium">Version</TableHead>
                   <TableHead class="px-3 py-2 font-medium text-right">Uptime</TableHead>
                   <TableHead class="px-3 py-2 font-medium" style={{ width: '180px' }}>CPU</TableHead>
@@ -194,15 +193,6 @@ export const ProxmoxNodesTable: Component<{
                             </span>
                           </div>
                         </TableCell>
-                        <TableCell class="px-3 py-2">
-                          <span
-                            class={`inline-flex items-center rounded-full px-2 py-0.5 text-[10px] font-semibold uppercase tracking-wide ${getStatusIndicatorBadgeToneClasses(
-                              indicator().variant,
-                            )}`}
-                          >
-                            {indicator().label}
-                          </span>
-                        </TableCell>
                         <TableCell class="px-3 py-2">
                           <Show
                             when={version()}
diff --git a/frontend-modern/src/features/proxmox/ProxmoxPageSurface.tsx b/frontend-modern/src/features/proxmox/ProxmoxPageSurface.tsx
index f67f36f36..518f23a13 100644
--- a/frontend-modern/src/features/proxmox/ProxmoxPageSurface.tsx
+++ b/frontend-modern/src/features/proxmox/ProxmoxPageSurface.tsx
@@ -147,7 +147,7 @@ export function ProxmoxPageSurface() {
                   showFilterToolbar
                   suppressPlatformFilter
                   forcedPlatform={PROXMOX_PLATFORM_FILTER}
-                  forcedGroupingMode="grouped"
+                  forcedGroupingMode="flat"
                 />
               </div>
             </Show>