From 3fd80977f67e9e4ff8dda13b8ba6d1d5bae6777e Mon Sep 17 00:00:00 2001
From: Taylor Ho <taylorkmho@gmail.com>
Date: Fri, 14 Aug 2026 10:43:59 -0700
Subject: [PATCH] Simplify workflow timing fields

Signed-off-by: Taylor Ho <taylorkmho@gmail.com>
---
 .../workflows/ui/WorkflowStepCard.tsx         | 36 ++++++-------------
 1 file changed, 11 insertions(+), 25 deletions(-)

diff --git a/desktop/src/features/workflows/ui/WorkflowStepCard.tsx b/desktop/src/features/workflows/ui/WorkflowStepCard.tsx
index e80ee4892..ac0f828e5 100644
--- a/desktop/src/features/workflows/ui/WorkflowStepCard.tsx
+++ b/desktop/src/features/workflows/ui/WorkflowStepCard.tsx
@@ -254,7 +254,7 @@ function StepConfigFields({
             disabled={disabled}
             fallbackSeconds={DEFAULT_APPROVAL_TIMEOUT_SECONDS}
             id={`${prefix}-timeout`}
-            label="Timeout (optional)"
+            label="Timeout"
             onChange={(timeout) => onUpdate({ ...step, timeout })}
             placeholder="24h"
             value={step.timeout ?? ""}
@@ -299,10 +299,6 @@ function StepConfigFields({
   }
 }
 
-function SectionHeading({ title }: { title: string }) {
-  return <h4 className="text-sm font-semibold text-foreground">{title}</h4>;
-}
-
 export function WorkflowStepCard({
   bare = false,
   channels = [],
@@ -356,7 +352,7 @@ export function WorkflowStepCard({
         </div>
       ) : null}
 
-      <section className="space-y-4 pb-5">
+      <section className="space-y-5 pb-5">
         <StepConfigFields
           channels={channels}
           disabled={disabled}
@@ -367,28 +363,18 @@ export function WorkflowStepCard({
           triggerType={triggerType}
           workflowChannelId={workflowChannelId}
         />
-      </section>
-
-      <section className="space-y-4 border-t border-border/50 py-5">
-        <SectionHeading title="Step timeout" />
-        <div className="space-y-2">
-          <WorkflowDurationField
-            disabled={disabled}
-            fallbackSeconds={DEFAULT_STEP_TIMEOUT_SECONDS}
-            id={`${prefix}-timeout-secs`}
-            label="Timeout (optional)"
-            onChange={(timeoutSecs) => onUpdate({ ...step, timeoutSecs })}
-            placeholder="5m"
-            value={step.timeoutSecs ?? ""}
-          />
-          <p className="text-xs text-muted-foreground">
-            Defaults to the workflow timeout.
-          </p>
-        </div>
+        <WorkflowDurationField
+          disabled={disabled}
+          fallbackSeconds={DEFAULT_STEP_TIMEOUT_SECONDS}
+          id={`${prefix}-timeout-secs`}
+          label="Timeout"
+          onChange={(timeoutSecs) => onUpdate({ ...step, timeoutSecs })}
+          placeholder="5m"
+          value={step.timeoutSecs ?? ""}
+        />
       </section>
 
       <section className="space-y-4 border-t border-border/50 pt-5">
-        <SectionHeading title="Step details" />
         <div className="space-y-1.5">
           <FieldLabel htmlFor={`${prefix}-name`}>Name (optional)</FieldLabel>
           <Input