From ef405af05ed3af8f7c91d1ef731f43bd2299a83a Mon Sep 17 00:00:00 2001
From: Waleed Latif <walif6@gmail.com>
Date: Tue, 21 Jul 2026 17:51:12 -0700
Subject: [PATCH] fix(workflow): align condition/router handles with their rows

---
 .../preview-workflow/components/block/block.tsx    |  2 +-
 packages/workflow-renderer/src/dimensions.ts       | 14 +++++++++-----
 .../src/workflow-block/sub-block-row-view.tsx      |  7 ++++++-
 3 files changed, 16 insertions(+), 7 deletions(-)

diff --git a/apps/sim/app/workspace/[workspaceId]/w/components/preview/components/preview-workflow/components/block/block.tsx b/apps/sim/app/workspace/[workspaceId]/w/components/preview/components/preview-workflow/components/block/block.tsx
index c28f99bbc8b..25dfd5d309e 100644
--- a/apps/sim/app/workspace/[workspaceId]/w/components/preview/components/preview-workflow/components/block/block.tsx
+++ b/apps/sim/app/workspace/[workspaceId]/w/components/preview/components/preview-workflow/components/block/block.tsx
@@ -146,7 +146,7 @@ const SubBlockRow = memo(function SubBlockRow({
   const displayValue = maskedValue || hydratedName || (isSelectorType && value ? '-' : value)
 
   return (
-    <div className='flex items-center gap-2'>
+    <div className='flex h-5 items-center gap-2'>
       <span
         className='min-w-0 truncate text-[var(--text-tertiary)] text-sm capitalize'
         title={title}
diff --git a/packages/workflow-renderer/src/dimensions.ts b/packages/workflow-renderer/src/dimensions.ts
index 89a15c31b15..736e95b7727 100644
--- a/packages/workflow-renderer/src/dimensions.ts
+++ b/packages/workflow-renderer/src/dimensions.ts
@@ -31,17 +31,21 @@ export const CONTAINER_DIMENSIONS = {
 } as const
 
 /**
- * Handle position constants - must match CSS in workflow-block.tsx and subflow-node.tsx
+ * Handle position constants - must match CSS in workflow-block-view.tsx,
+ * sub-block-row-view.tsx, and subflow-node.tsx
  */
 export const HANDLE_POSITIONS = {
   /** Default Y offset from block top for source/target handles */
   DEFAULT_Y_OFFSET: 20,
   /** Error handle offset from block bottom */
   ERROR_BOTTOM_OFFSET: 17,
-  /** Condition handle starting Y offset */
-  CONDITION_START_Y: 60,
-  /** Height per condition row */
-  CONDITION_ROW_HEIGHT: 29,
+  /**
+   * Y of the first condition-row handle: 40px header + 1px divider +
+   * 8px content padding + half of the 20px row
+   */
+  CONDITION_START_Y: 59,
+  /** Row pitch: 20px row height (h-5) + 8px flex gap (gap-2) */
+  CONDITION_ROW_HEIGHT: 28,
   /** Subflow start handle Y offset (header 50px + pill offset 16px + pill center 14px) */
   SUBFLOW_START_Y_OFFSET: 80,
 } as const
diff --git a/packages/workflow-renderer/src/workflow-block/sub-block-row-view.tsx b/packages/workflow-renderer/src/workflow-block/sub-block-row-view.tsx
index 7be0b0bb7b6..87c4ac824b4 100644
--- a/packages/workflow-renderer/src/workflow-block/sub-block-row-view.tsx
+++ b/packages/workflow-renderer/src/workflow-block/sub-block-row-view.tsx
@@ -19,10 +19,15 @@ export interface SubBlockRowViewProps {
 /**
  * Pure renderer for a collapsed block's subblock summary row: a capitalized
  * title and its resolved display value.
+ *
+ * The fixed `h-5` row height is part of the handle-position contract —
+ * `HANDLE_POSITIONS.CONDITION_ROW_HEIGHT` in dimensions.ts assumes a 20px row
+ * plus the container's 8px gap, so condition/router source handles align with
+ * their rows.
  */
 export function SubBlockRowView({ title, displayValue, isMonospace }: SubBlockRowViewProps) {
   return (
-    <div className='flex items-center gap-2'>
+    <div className='flex h-5 items-center gap-2'>
       <OverflowSpan
         value={title}
         className='min-w-0 truncate text-[var(--text-tertiary)] text-sm capitalize'