Skip to content

Commit 7b7be2b

Browse files
feat: add tooltip to status in cols (#660)
1 parent 656c37e commit 7b7be2b

File tree

2 files changed

+44
-13
lines changed

2 files changed

+44
-13
lines changed

src/app/pipelines/[namespace]/columns.tsx

Lines changed: 20 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,13 @@ import { PipelineRun, PipelineRunBody } from "@/types/pipeline-runs";
99
import { Stack } from "@/types/stack";
1010
import { User } from "@/types/user";
1111
import { ColumnDef } from "@tanstack/react-table";
12-
import { Tag } from "@zenml-io/react-component-library";
12+
import {
13+
Tag,
14+
Tooltip,
15+
TooltipContent,
16+
TooltipProvider,
17+
TooltipTrigger
18+
} from "@zenml-io/react-component-library";
1319
import { Link } from "react-router-dom";
1420

1521
export function getPipelineDetailColumns(): ColumnDef<PipelineRun>[] {
@@ -33,11 +39,20 @@ export function getPipelineDetailColumns(): ColumnDef<PipelineRun>[] {
3339
<div className="group/copybutton flex items-center gap-2">
3440
<RunIcon className={`h-5 w-5 shrink-0 ${getExecutionStatusColor(status)}`} />
3541
<div>
36-
<Link to={routes.runs.detail(id)} className="flex items-center gap-1">
37-
<h2 className="text-text-md font-semibold">{name}</h2>
38-
<ExecutionStatusIcon status={status} />
42+
<div className="flex items-center gap-1">
43+
<Link to={routes.runs.detail(id)} className="flex items-center gap-1">
44+
<h2 className="text-text-md font-semibold">{name}</h2>
45+
</Link>
46+
<TooltipProvider>
47+
<Tooltip>
48+
<TooltipTrigger className="hover:text-theme-text-brand hover:underline">
49+
<ExecutionStatusIcon status={status} />
50+
</TooltipTrigger>
51+
<TooltipContent className="z-20 capitalize">{status}</TooltipContent>
52+
</Tooltip>
53+
</TooltipProvider>
3954
<CopyButton copyText={name} />
40-
</Link>
55+
</div>
4156
<Link to={routes.runs.detail(id)} className="flex items-center gap-1">
4257
<p className="text-text-xs text-theme-text-secondary">{id.split("-")[0]}</p>
4358
<CopyButton copyText={id} />

src/app/pipelines/columns.tsx

Lines changed: 24 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,14 @@ import { routes } from "@/router/routes";
66
import { ExecutionStatus } from "@/types/pipeline-runs";
77
import { Pipeline } from "@/types/pipelines";
88
import { ColumnDef } from "@tanstack/react-table";
9-
import { Tag, TagProps } from "@zenml-io/react-component-library";
9+
import {
10+
Tag,
11+
TagProps,
12+
Tooltip,
13+
TooltipContent,
14+
TooltipProvider,
15+
TooltipTrigger
16+
} from "@zenml-io/react-component-library";
1017
import { Link } from "react-router-dom";
1118

1219
export function getPipelineColumns(): ColumnDef<Pipeline>[] {
@@ -24,14 +31,23 @@ export function getPipelineColumns(): ColumnDef<Pipeline>[] {
2431
return (
2532
<div className="group/copybutton flex items-center gap-2">
2633
<PipelineIcon className={`h-5 w-5 ${getExecutionStatusColor(status)}`} />
27-
<Link
28-
to={routes.pipelines.namespace(encodeURIComponent(name))}
29-
className="flex items-center gap-1"
30-
>
31-
<h2 className="text-text-md font-semibold">{name}</h2>
32-
<ExecutionStatusIcon status={status} />
34+
<div className="flex items-center gap-1">
35+
<Link
36+
to={routes.pipelines.namespace(encodeURIComponent(name))}
37+
className="flex items-center gap-1"
38+
>
39+
<h2 className="text-text-md font-semibold">{name}</h2>
40+
</Link>
41+
<TooltipProvider>
42+
<Tooltip>
43+
<TooltipTrigger className="hover:text-theme-text-brand hover:underline">
44+
<ExecutionStatusIcon status={status} />
45+
</TooltipTrigger>
46+
<TooltipContent className="z-20 capitalize">{status}</TooltipContent>
47+
</Tooltip>
48+
</TooltipProvider>
3349
<CopyButton copyText={name} />
34-
</Link>
50+
</div>
3551
</div>
3652
);
3753
}

0 commit comments

Comments
 (0)