From f83f01417cd34e414d48ac09a0cd400a233585b1 Mon Sep 17 00:00:00 2001 From: SaelixCode Date: Fri, 3 Jul 2026 15:21:35 -0400 Subject: [PATCH] Revert "fix: standardize all tooltips to match sidebar RowTooltip design" This reverts commit 9e1d97d39e294a10ff06ec03e94aa751728d1342. --- frontend/src/components/NodeManager.tsx | 12 ++++++------ frontend/src/components/StackAlertSheet.tsx | 2 +- frontend/src/components/TopBar.tsx | 2 +- .../src/components/fleet/FleetConfiguration.tsx | 2 +- frontend/src/components/fleet/FleetTopology.tsx | 14 ++++++-------- frontend/src/components/sidebar/SidebarActions.tsx | 4 ++-- .../components/sidebar/__tests__/StackRow.test.tsx | 2 +- 7 files changed, 18 insertions(+), 20 deletions(-) diff --git a/frontend/src/components/NodeManager.tsx b/frontend/src/components/NodeManager.tsx index 07e991bc..6ec431b8 100644 --- a/frontend/src/components/NodeManager.tsx +++ b/frontend/src/components/NodeManager.tsx @@ -333,7 +333,7 @@ export function NodeManager() { - Default Node + Default Node )} @@ -395,7 +395,7 @@ export function NodeManager() { - {new Date(summary.next_run_at).toLocaleString()} + {new Date(summary.next_run_at).toLocaleString()} @@ -450,7 +450,7 @@ export function NodeManager() { - View Schedules + View Schedules @@ -469,7 +469,7 @@ export function NodeManager() { - Test Connection + Test Connection )} @@ -488,7 +488,7 @@ export function NodeManager() { - Edit Node + Edit Node )} @@ -507,7 +507,7 @@ export function NodeManager() { - Delete Node + Delete Node )} diff --git a/frontend/src/components/StackAlertSheet.tsx b/frontend/src/components/StackAlertSheet.tsx index c3977a1e..6d3032a3 100644 --- a/frontend/src/components/StackAlertSheet.tsx +++ b/frontend/src/components/StackAlertSheet.tsx @@ -384,7 +384,7 @@ function AlertsTab({ stackName }: { stackName: string }) { - The system resource or metric to monitor. Select from CPU, Memory, Network I/O, or Restarts. +

The system resource or metric to monitor. Select from CPU, Memory, Network I/O, or Restarts.

diff --git a/frontend/src/components/TopBar.tsx b/frontend/src/components/TopBar.tsx index 8aff520e..aeba5256 100644 --- a/frontend/src/components/TopBar.tsx +++ b/frontend/src/components/TopBar.tsx @@ -90,7 +90,7 @@ export function TopBar({ ) : ( {button} - {label} + {label} ); })} diff --git a/frontend/src/components/fleet/FleetConfiguration.tsx b/frontend/src/components/fleet/FleetConfiguration.tsx index 214c12b2..7d7bcc81 100644 --- a/frontend/src/components/fleet/FleetConfiguration.tsx +++ b/frontend/src/components/fleet/FleetConfiguration.tsx @@ -84,7 +84,7 @@ function PolicySyncRow({ state }: { state: PolicySyncState }) { {state.kind === 'paused' ? 'paused' : 'degraded'} - {tooltip} + {tooltip} diff --git a/frontend/src/components/fleet/FleetTopology.tsx b/frontend/src/components/fleet/FleetTopology.tsx index 660a1280..8d091ce4 100644 --- a/frontend/src/components/fleet/FleetTopology.tsx +++ b/frontend/src/components/fleet/FleetTopology.tsx @@ -122,7 +122,7 @@ function FleetNodeCard({ data, selected }: { data: FleetNodeData; selected?: boo - {node.cordonedReason || 'Cordoned: scheduling paused.'} + {node.cordonedReason || 'Cordoned: scheduling paused.'} @@ -141,11 +141,9 @@ function FleetNodeCard({ data, selected }: { data: FleetNodeData; selected?: boo - - {node.pilotLastSeen - ? `Pilot heartbeat ${formatRelative(node.pilotLastSeen)}` - : 'Pilot heartbeat stale'} - + {node.pilotLastSeen + ? `Pilot heartbeat ${formatRelative(node.pilotLastSeen)}` + : 'Pilot heartbeat stale'} @@ -181,7 +179,7 @@ function FleetNodeCard({ data, selected }: { data: FleetNodeData; selected?: boo - {labels.slice(MAX_INLINE_LABELS).join(', ')} + {labels.slice(MAX_INLINE_LABELS).join(', ')} @@ -262,7 +260,7 @@ function TopologyToolbar({ - {spec.description} + {spec.description} diff --git a/frontend/src/components/sidebar/SidebarActions.tsx b/frontend/src/components/sidebar/SidebarActions.tsx index 12c9e460..478767ed 100644 --- a/frontend/src/components/sidebar/SidebarActions.tsx +++ b/frontend/src/components/sidebar/SidebarActions.tsx @@ -30,7 +30,7 @@ export function SidebarActions({ createStackSlot, onScan, isScanning, bulkMode, - Bulk mode (B) +

Bulk mode (B)

@@ -50,7 +50,7 @@ export function SidebarActions({ createStackSlot, onScan, isScanning, bulkMode, - Scan stacks folder +

Scan stacks folder

diff --git a/frontend/src/components/sidebar/__tests__/StackRow.test.tsx b/frontend/src/components/sidebar/__tests__/StackRow.test.tsx index 15b642dc..a200979c 100644 --- a/frontend/src/components/sidebar/__tests__/StackRow.test.tsx +++ b/frontend/src/components/sidebar/__tests__/StackRow.test.tsx @@ -43,7 +43,7 @@ describe('StackRow', () => { }); it('wraps the partial pill in a hover tooltip', () => { - render(); + const { container } = render(); const trigger = screen.getByText('PT'); // Radix TooltipTrigger adds data-state to the wrapped element. expect(trigger.getAttribute('data-state')).toBe('closed');