mirror of
https://github.com/rcourtman/Pulse.git
synced 2026-10-03 20:53:09 +00:00
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.
This commit is contained in:
@@ -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()}
|
||||
|
||||
@@ -147,7 +147,7 @@ export function ProxmoxPageSurface() {
|
||||
showFilterToolbar
|
||||
suppressPlatformFilter
|
||||
forcedPlatform={PROXMOX_PLATFORM_FILTER}
|
||||
forcedGroupingMode="grouped"
|
||||
forcedGroupingMode="flat"
|
||||
/>
|
||||
</div>
|
||||
</Show>
|
||||
|
||||
Reference in New Issue
Block a user