diff --git a/src/components/Editor/PipelineDetails.tsx b/src/components/Editor/PipelineDetails.tsx index ef80203ab..e26529a56 100644 --- a/src/components/Editor/PipelineDetails.tsx +++ b/src/components/Editor/PipelineDetails.tsx @@ -6,7 +6,7 @@ import { useValidationIssueNavigation } from "@/components/Editor/hooks/useValid import { CopyText } from "@/components/shared/CopyText/CopyText"; import { Button } from "@/components/ui/button"; import { Icon } from "@/components/ui/icon"; -import { InlineStack } from "@/components/ui/layout"; +import { BlockStack, InlineStack } from "@/components/ui/layout"; import useToastNotification from "@/hooks/useToastNotification"; import { useComponentSpec } from "@/providers/ComponentSpecProvider"; import { useContextPanel } from "@/providers/ContextPanelProvider"; @@ -128,25 +128,22 @@ const PipelineDetails = () => { const annotations = componentSpec.metadata?.annotations || {}; return ( -
- {/* Header */} -
- - {componentSpec.name ?? "Unnamed Pipeline"} - + + {componentSpec.name ?? "Unnamed Pipeline"} + + -
- -
-
+ {/* General Metadata */}
@@ -326,7 +323,7 @@ const PipelineDetails = () => { onIssueSelect={handleIssueClick} />
-
+ ); }; diff --git a/src/components/Editor/RenamePipeline.tsx b/src/components/Editor/RenamePipeline.tsx index f35940125..a64c7c247 100644 --- a/src/components/Editor/RenamePipeline.tsx +++ b/src/components/Editor/RenamePipeline.tsx @@ -1,7 +1,7 @@ import { useLocation, useNavigate } from "@tanstack/react-router"; import { Edit3 } from "lucide-react"; -import { Button } from "@/components/ui/button"; +import TooltipButton from "@/components/shared/Buttons/TooltipButton"; import useToastNotification from "@/hooks/useToastNotification"; import { useComponentSpec } from "@/providers/ComponentSpecProvider"; import { APP_ROUTES } from "@/routes/router"; @@ -48,9 +48,9 @@ const RenamePipeline = () => { return ( + - + } title="Name Pipeline" description="Unsaved pipeline changes will be lost." diff --git a/src/components/PipelineRun/RunDetails.tsx b/src/components/PipelineRun/RunDetails.tsx index 7be9ce59c..c92c5ae8f 100644 --- a/src/components/PipelineRun/RunDetails.tsx +++ b/src/components/PipelineRun/RunDetails.tsx @@ -1,7 +1,9 @@ import { Frown } from "lucide-react"; import { CopyText } from "@/components/shared/CopyText/CopyText"; +import { BlockStack, InlineStack } from "@/components/ui/layout"; import { Spinner } from "@/components/ui/spinner"; +import { Text } from "@/components/ui/typography"; import { useCheckComponentSpecFromPath } from "@/hooks/useCheckComponentSpecFromPath"; import { useUserDetails } from "@/hooks/useUserDetails"; import { useBackend } from "@/providers/BackendProvider"; @@ -15,7 +17,7 @@ import { } from "@/services/executionService"; import { InfoBox } from "../shared/InfoBox"; -import { StatusBar, StatusIcon, StatusText } from "../shared/Status"; +import { StatusBar, StatusText } from "../shared/Status"; import { TaskImplementation } from "../shared/TaskDetails"; import { CancelPipelineRunButton } from "./components/CancelPipelineRunButton"; import { ClonePipelineButton } from "./components/ClonePipelineButton"; @@ -82,13 +84,26 @@ export const RunDetails = () => { const annotations = componentSpec.metadata?.annotations || {}; return ( -
-
- - {componentSpec.name ?? "Unnamed Pipeline"} - - -
+ + + {componentSpec.name ?? "Unnamed Pipeline"} + + + + + {canAccessEditorSpec && componentSpec.name && ( + + )} + + {isInProgress && isRunCreator && ( + + )} + {isComplete && } + {metadata && (
@@ -124,24 +139,6 @@ export const RunDetails = () => {
)} -
-
- - {canAccessEditorSpec && componentSpec.name && ( - - )} - - {isInProgress && isRunCreator && ( - - )} - {isComplete && } -
-
- {componentSpec.description && (

Description

@@ -151,15 +148,15 @@ export const RunDetails = () => {
)} -
-
-

Status: {runStatus}

+ + + + Status: {runStatus} + -
-
- -
-
+ + +
{Object.keys(annotations).length > 0 && (
@@ -175,65 +172,55 @@ export const RunDetails = () => {
)} -
+

Artifacts

-
-
+
+

Inputs

{componentSpec.inputs && componentSpec.inputs.length > 0 ? ( -
    +
    {componentSpec.inputs.map((input) => ( -
  • - {input.name} - {input.type && ( - - ( - {typeof input.type === "string" ? input.type : "object"} - ) - - )} - {input.description && ( -
    - {input.description} -
    - )} -
  • +
    +
    + {input.name} +
    +
    + {typeof input.type === "string" ? input.type : "object"} +
    +
    ))} -
+
) : (
No inputs
)}
-
+

Outputs

{componentSpec.outputs && componentSpec.outputs.length > 0 ? ( -
    +
    {componentSpec.outputs.map((output) => ( -
  • - {output.name} - {output.type && ( - - ( - {typeof output.type === "string" - ? output.type - : "object"} - ) - - )} - {output.description && ( -
    - {output.description} -
    - )} -
  • +
    +
    + {output.name} +
    +
    + {typeof output.type === "string" ? output.type : "object"} +
    +
    ))} -
+
) : (
No outputs
)}
-
+ ); }; diff --git a/src/components/ui/layout.tsx b/src/components/ui/layout.tsx index 49c50d850..120fa991a 100644 --- a/src/components/ui/layout.tsx +++ b/src/components/ui/layout.tsx @@ -32,6 +32,8 @@ const blockStackVariants = cva("flex flex-col w-full", { "2": "gap-2", "3": "gap-3", "4": "gap-4", + "5": "gap-5", + "6": "gap-6", "8": "gap-8", }, }, @@ -101,6 +103,9 @@ const inlineStackVariants = cva("flex flex-row", { "2": "gap-2", "3": "gap-3", "4": "gap-4", + "5": "gap-5", + "6": "gap-6", + "8": "gap-8", }, wrap: { wrap: "flex-wrap",