From b2f7252b27a8a9fa9212ad17a3df5c0b65bd6b14 Mon Sep 17 00:00:00 2001
From: Dotta <bippadotta@protonmail.com>
Date: Tue, 17 Mar 2026 09:49:12 -0500
Subject: [PATCH] Fix empty space in new issue pane when workspaces disabled

The execution workspace section wrapper div was rendered whenever a
project was selected, even when experimental workspaces were off.
The empty div's py-3 padding caused a visible layout bump. Now the
entire block only renders when currentProjectSupportsExecutionWorkspace
is true, and the redundant inner conditional is removed.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 ui/src/components/NewIssueDialog.tsx | 74 ++++++++++++++--------------
 1 file changed, 36 insertions(+), 38 deletions(-)

diff --git a/ui/src/components/NewIssueDialog.tsx b/ui/src/components/NewIssueDialog.tsx
index a734fbd5..ad8efb67 100644
--- a/ui/src/components/NewIssueDialog.tsx
+++ b/ui/src/components/NewIssueDialog.tsx
@@ -1117,51 +1117,49 @@ export function NewIssueDialog() {
           </div>
         </div>
 
-        {currentProject && (
+        {currentProject && currentProjectSupportsExecutionWorkspace && (
           <div className="px-4 py-3 shrink-0 space-y-2">
-            {currentProjectSupportsExecutionWorkspace && (
-              <div className="space-y-1.5">
-                <div className="text-xs font-medium">Execution workspace</div>
-                <div className="text-[11px] text-muted-foreground">
-                  Control whether this issue runs in the shared workspace, a new isolated workspace, or an existing one.
-                </div>
+            <div className="space-y-1.5">
+              <div className="text-xs font-medium">Execution workspace</div>
+              <div className="text-[11px] text-muted-foreground">
+                Control whether this issue runs in the shared workspace, a new isolated workspace, or an existing one.
+              </div>
+              <select
+                className="w-full rounded border border-border bg-transparent px-2 py-1.5 text-xs outline-none"
+                value={executionWorkspaceMode}
+                onChange={(e) => {
+                  setExecutionWorkspaceMode(e.target.value);
+                  if (e.target.value !== "reuse_existing") {
+                    setSelectedExecutionWorkspaceId("");
+                  }
+                }}
+              >
+                {EXECUTION_WORKSPACE_MODES.map((option) => (
+                  <option key={option.value} value={option.value}>
+                    {option.label}
+                  </option>
+                ))}
+              </select>
+              {executionWorkspaceMode === "reuse_existing" && (
                 <select
                   className="w-full rounded border border-border bg-transparent px-2 py-1.5 text-xs outline-none"
-                  value={executionWorkspaceMode}
-                  onChange={(e) => {
-                    setExecutionWorkspaceMode(e.target.value);
-                    if (e.target.value !== "reuse_existing") {
-                      setSelectedExecutionWorkspaceId("");
-                    }
-                  }}
+                  value={selectedExecutionWorkspaceId}
+                  onChange={(e) => setSelectedExecutionWorkspaceId(e.target.value)}
                 >
-                  {EXECUTION_WORKSPACE_MODES.map((option) => (
-                    <option key={option.value} value={option.value}>
-                      {option.label}
+                  <option value="">Choose an existing workspace</option>
+                  {deduplicatedReusableWorkspaces.map((workspace) => (
+                    <option key={workspace.id} value={workspace.id}>
+                      {workspace.name} · {workspace.status} · {workspace.branchName ?? workspace.cwd ?? workspace.id.slice(0, 8)}
                     </option>
                   ))}
                 </select>
-                {executionWorkspaceMode === "reuse_existing" && (
-                  <select
-                    className="w-full rounded border border-border bg-transparent px-2 py-1.5 text-xs outline-none"
-                    value={selectedExecutionWorkspaceId}
-                    onChange={(e) => setSelectedExecutionWorkspaceId(e.target.value)}
-                  >
-                    <option value="">Choose an existing workspace</option>
-                    {deduplicatedReusableWorkspaces.map((workspace) => (
-                      <option key={workspace.id} value={workspace.id}>
-                        {workspace.name} · {workspace.status} · {workspace.branchName ?? workspace.cwd ?? workspace.id.slice(0, 8)}
-                      </option>
-                    ))}
-                  </select>
-                )}
-                {executionWorkspaceMode === "reuse_existing" && selectedReusableExecutionWorkspace && (
-                  <div className="text-[11px] text-muted-foreground">
-                    Reusing {selectedReusableExecutionWorkspace.name} from {selectedReusableExecutionWorkspace.branchName ?? selectedReusableExecutionWorkspace.cwd ?? "existing execution workspace"}.
-                  </div>
-                )}
-              </div>
-            )}
+              )}
+              {executionWorkspaceMode === "reuse_existing" && selectedReusableExecutionWorkspace && (
+                <div className="text-[11px] text-muted-foreground">
+                  Reusing {selectedReusableExecutionWorkspace.name} from {selectedReusableExecutionWorkspace.branchName ?? selectedReusableExecutionWorkspace.cwd ?? "existing execution workspace"}.
+                </div>
+              )}
+            </div>
           </div>
         )}