Revert "fix: standardize all tooltips to match sidebar RowTooltip design"

This reverts commit 9e1d97d39e.
This commit is contained in:
SaelixCode
2026-07-03 15:21:35 -04:00
parent 9e1d97d39e
commit f83f01417c
7 changed files with 18 additions and 20 deletions
+6 -6
View File
@@ -333,7 +333,7 @@ export function NodeManager() {
<TooltipTrigger>
<Star className="w-4 h-4 text-yellow-500 fill-yellow-500" />
</TooltipTrigger>
<TooltipContent><span className="font-mono text-xs tabular-nums text-stat-value">Default Node</span></TooltipContent>
<TooltipContent>Default Node</TooltipContent>
</Tooltip>
</TooltipProvider>
)}
@@ -395,7 +395,7 @@ export function NodeManager() {
</span>
</TooltipTrigger>
<TooltipContent>
<span className="font-mono text-xs tabular-nums text-stat-value">{new Date(summary.next_run_at).toLocaleString()}</span>
{new Date(summary.next_run_at).toLocaleString()}
</TooltipContent>
</Tooltip>
</TooltipProvider>
@@ -450,7 +450,7 @@ export function NodeManager() {
<Calendar className="w-4 h-4" strokeWidth={1.5} />
</Button>
</TooltipTrigger>
<TooltipContent><span className="font-mono text-xs tabular-nums text-stat-value">View Schedules</span></TooltipContent>
<TooltipContent>View Schedules</TooltipContent>
</Tooltip>
</TooltipProvider>
@@ -469,7 +469,7 @@ export function NodeManager() {
<Wifi className={`w-4 h-4 ${testing === node.id ? 'animate-pulse' : ''}`} />
</Button>
</TooltipTrigger>
<TooltipContent><span className="font-mono text-xs tabular-nums text-stat-value">Test Connection</span></TooltipContent>
<TooltipContent>Test Connection</TooltipContent>
</Tooltip>
</TooltipProvider>
)}
@@ -488,7 +488,7 @@ export function NodeManager() {
<Pencil className="w-4 h-4" />
</Button>
</TooltipTrigger>
<TooltipContent><span className="font-mono text-xs tabular-nums text-stat-value">Edit Node</span></TooltipContent>
<TooltipContent>Edit Node</TooltipContent>
</Tooltip>
</TooltipProvider>
)}
@@ -507,7 +507,7 @@ export function NodeManager() {
<Trash2 className="w-4 h-4" />
</Button>
</TooltipTrigger>
<TooltipContent><span className="font-mono text-xs tabular-nums text-stat-value">Delete Node</span></TooltipContent>
<TooltipContent>Delete Node</TooltipContent>
</Tooltip>
</TooltipProvider>
)}
+1 -1
View File
@@ -384,7 +384,7 @@ function AlertsTab({ stackName }: { stackName: string }) {
<HelpCircle className="h-4 w-4 text-muted-foreground cursor-help" strokeWidth={1.5} />
</TooltipTrigger>
<TooltipContent>
<span className="font-mono text-xs tabular-nums text-stat-value max-w-[200px]">The system resource or metric to monitor. Select from CPU, Memory, Network I/O, or Restarts.</span>
<p className="max-w-[200px] text-sm">The system resource or metric to monitor. Select from CPU, Memory, Network I/O, or Restarts.</p>
</TooltipContent>
</Tooltip>
</div>
+1 -1
View File
@@ -90,7 +90,7 @@ export function TopBar({
) : (
<Tooltip key={value}>
<TooltipTrigger asChild>{button}</TooltipTrigger>
<TooltipContent side="bottom"><span className="font-mono text-xs tabular-nums text-stat-value">{label}</span></TooltipContent>
<TooltipContent side="bottom">{label}</TooltipContent>
</Tooltip>
);
})}
@@ -84,7 +84,7 @@ function PolicySyncRow({ state }: { state: PolicySyncState }) {
{state.kind === 'paused' ? 'paused' : 'degraded'}
</span>
</TooltipTrigger>
<TooltipContent><span className="font-mono text-xs tabular-nums text-stat-value">{tooltip}</span></TooltipContent>
<TooltipContent>{tooltip}</TooltipContent>
</Tooltip>
</TooltipProvider>
</div>
@@ -122,7 +122,7 @@ function FleetNodeCard({ data, selected }: { data: FleetNodeData; selected?: boo
</div>
</TooltipTrigger>
<TooltipContent side="top">
<span className="font-mono text-xs tabular-nums text-stat-value">{node.cordonedReason || 'Cordoned: scheduling paused.'}</span>
{node.cordonedReason || 'Cordoned: scheduling paused.'}
</TooltipContent>
</Tooltip>
</TooltipProvider>
@@ -141,11 +141,9 @@ function FleetNodeCard({ data, selected }: { data: FleetNodeData; selected?: boo
<Clock className="h-3 w-3 text-warning" strokeWidth={2} aria-label="Pilot heartbeat stale" />
</TooltipTrigger>
<TooltipContent side="top">
<span className="font-mono text-xs tabular-nums text-stat-value">
{node.pilotLastSeen
? `Pilot heartbeat ${formatRelative(node.pilotLastSeen)}`
: 'Pilot heartbeat stale'}
</span>
{node.pilotLastSeen
? `Pilot heartbeat ${formatRelative(node.pilotLastSeen)}`
: 'Pilot heartbeat stale'}
</TooltipContent>
</Tooltip>
</TooltipProvider>
@@ -181,7 +179,7 @@ function FleetNodeCard({ data, selected }: { data: FleetNodeData; selected?: boo
</span>
</TooltipTrigger>
<TooltipContent side="top">
<span className="font-mono text-xs tabular-nums text-stat-value">{labels.slice(MAX_INLINE_LABELS).join(', ')}</span>
{labels.slice(MAX_INLINE_LABELS).join(', ')}
</TooltipContent>
</Tooltip>
</TooltipProvider>
@@ -262,7 +260,7 @@ function TopologyToolbar({
</button>
</TooltipTrigger>
<TooltipContent side="bottom">
<span className="font-mono text-xs tabular-nums text-stat-value">{spec.description}</span>
{spec.description}
</TooltipContent>
</Tooltip>
</TooltipProvider>
@@ -30,7 +30,7 @@ export function SidebarActions({ createStackSlot, onScan, isScanning, bulkMode,
</Button>
</TooltipTrigger>
<TooltipContent side="bottom">
<span className="font-mono text-xs tabular-nums text-stat-value">Bulk mode (B)</span>
<p>Bulk mode (B)</p>
</TooltipContent>
</Tooltip>
</TooltipProvider>
@@ -50,7 +50,7 @@ export function SidebarActions({ createStackSlot, onScan, isScanning, bulkMode,
</Button>
</TooltipTrigger>
<TooltipContent side="bottom">
<span className="font-mono text-xs tabular-nums text-stat-value">Scan stacks folder</span>
<p>Scan stacks folder</p>
</TooltipContent>
</Tooltip>
</TooltipProvider>
@@ -43,7 +43,7 @@ describe('StackRow', () => {
});
it('wraps the partial pill in a hover tooltip', () => {
render(<StackRow {...base({ status: 'partial', running: 3, total: 5 })} />);
const { container } = render(<StackRow {...base({ status: 'partial', running: 3, total: 5 })} />);
const trigger = screen.getByText('PT');
// Radix TooltipTrigger adds data-state to the wrapped element.
expect(trigger.getAttribute('data-state')).toBe('closed');