Files
supabase/apps/studio/components/interfaces/ExplainVisualizer/ExplainVisualizer.RowCountIndicator.tsx
T
Joshen Lim 09a1f431bc Mass update for all lucide icon imports to use the component with Icon suffix (#51659)
## Context

This is largely just a mechanical change here:
Opting to use lucide icons with the `Icon` suffix across all icon
imports within `studio`
- so that it’s clearer which components are icons + reduces the
possibility of name collision with our own components
- e.g <User /> vs <UserIcon />

Also added a eslint rule for this:
<img width="1069" height="167" alt="image"
src="https://github.com/user-attachments/assets/7f36f443-5ca2-4bf6-8d22-86e774513bfc"
/>

And updated `AGENTS.md` as such:
```
Icons come from `lucide-react`, always imported with the `Icon` suffix (`import { CheckIcon, Trash2Icon } from 'lucide-react'`) — never the bare names (`Check`) or `Check as CheckIcon` aliases (lint-enforced).
```
2026-10-14 12:56:55 +02:00

95 lines
3.5 KiB
TypeScript

import { ArrowRightIcon } from 'lucide-react'
import { Tooltip, TooltipContent, TooltipTrigger } from 'ui'
interface RowCountIndicatorProps {
actualRows?: number
estimatedRows?: number
rowsRemovedByFilter?: number
}
function formatRowCount(rows: number | undefined): string {
if (rows === undefined) return '-'
return rows.toLocaleString()
}
export function RowCountIndicator({
actualRows,
estimatedRows,
rowsRemovedByFilter,
}: RowCountIndicatorProps) {
const hasActualRows = actualRows !== undefined
const hasEstimatedRows = estimatedRows !== undefined
const hasFilterData = rowsRemovedByFilter !== undefined && hasActualRows
const totalRowsScanned = hasFilterData ? actualRows + rowsRemovedByFilter : undefined
// Show filter flow: scanned → filtered out → remaining
if (hasFilterData && totalRowsScanned !== undefined) {
return (
<Tooltip>
<TooltipTrigger asChild>
<div className="flex items-center gap-1.5 cursor-help">
<span className="text-foreground-light">{formatRowCount(totalRowsScanned)}</span>
<ArrowRightIcon size={10} className="text-foreground-muted" />
<span className="text-destructive-600 font-medium">
-{formatRowCount(rowsRemovedByFilter)}
</span>
<ArrowRightIcon size={10} className="text-foreground-muted" />
<span className="text-primary font-medium">
{formatRowCount(actualRows)} {actualRows === 1 ? 'row' : 'rows'}
</span>
</div>
</TooltipTrigger>
<TooltipContent side="top" className="max-w-xs font-sans">
<p className="font-medium">Filter applied</p>
<p className="text-foreground-lighter text-xs mt-1">
{formatRowCount(totalRowsScanned)} rows were scanned,{' '}
{formatRowCount(rowsRemovedByFilter)} were filtered out, leaving{' '}
{formatRowCount(actualRows)} rows. Consider adding an index to reduce rows scanned.
</p>
</TooltipContent>
</Tooltip>
)
}
// Simple row count with actual rows
if (hasActualRows) {
return (
<Tooltip>
<TooltipTrigger asChild>
<span className="text-foreground-light cursor-help">
{formatRowCount(actualRows)} {actualRows === 1 ? 'row' : 'rows'}
</span>
</TooltipTrigger>
<TooltipContent side="top" className="max-w-xs font-sans">
<p className="font-medium">Rows returned</p>
<p className="text-foreground-lighter text-xs mt-1">
This operation processed and returned {formatRowCount(actualRows)} rows.
</p>
</TooltipContent>
</Tooltip>
)
}
// Only estimated rows available (EXPLAIN without ANALYZE)
if (hasEstimatedRows) {
return (
<Tooltip>
<TooltipTrigger asChild>
<span className="text-foreground-muted cursor-help">
est. {formatRowCount(estimatedRows)} {estimatedRows === 1 ? 'row' : 'rows'}
</span>
</TooltipTrigger>
<TooltipContent side="top" className="max-w-xs font-sans">
<p className="font-medium">Estimated rows</p>
<p className="text-foreground-lighter text-xs mt-1">
The planner estimates this operation will return {formatRowCount(estimatedRows)} rows.
Run with EXPLAIN ANALYZE to see actual row counts.
</p>
</TooltipContent>
</Tooltip>
)
}
return <span className="text-foreground-muted">-</span>
}