From 413afde06d541695975ed389077135926826a6da Mon Sep 17 00:00:00 2001
From: Chen Li <chenli@gmail.com>
Date: Sat, 16 May 2026 17:38:35 -0700
Subject: [PATCH] fix(frontend): remove the hover popover next to the Run
 button
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The "Execution Settings" hover popover next to the Run button — with an
execution-name input and an email-notification toggle — was unnecessary
UX clutter. The runWorkflow() handler already falls back to "Untitled
Execution" when no explicit name is provided, so removing the popover
leaves the Run button's behavior unchanged.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .../component/menu/menu.component.html        | 21 -------------------
 1 file changed, 21 deletions(-)

diff --git a/frontend/src/app/workspace/component/menu/menu.component.html b/frontend/src/app/workspace/component/menu/menu.component.html
index a21e4d56429..ab97a3c59c8 100644
--- a/frontend/src/app/workspace/component/menu/menu.component.html
+++ b/frontend/src/app/workspace/component/menu/menu.component.html
@@ -391,11 +391,6 @@
         </button>
         <button
           (click)="onClickRunHandler()"
-          nz-popover
-          nzPopoverTitle="Execution Settings"
-          [nzPopoverTrigger]="'hover'"
-          [nzPopoverContent]="executionSettings"
-          nzPopoverPlacement="bottom"
           [disabled]="runDisable || (!workflowWebsocketService.isConnected && computingUnitStatus !== ComputingUnitState.NoComputingUnit) || displayParticularWorkflowVersion || selectedComputingUnit?.accessPrivilege !== Privilege.WRITE"
           id="run-button"
           nz-button
@@ -417,22 +412,6 @@
             nz-icon
             nzType="database"></i>
         </button>
-        <ng-template #executionSettings>
-          <div style="display: flex; flex-direction: column; gap: 10px">
-            <input
-              [(ngModel)]="currentExecutionName"
-              placeholder="Untitled Execution"
-              [disabled]="!isWorkflowModifiable" />
-            <div style="display: flex; align-items: center; gap: 10px">
-              <span>Email Notification</span>
-              <nz-switch
-                [(ngModel)]="this.config.env.workflowEmailNotificationEnabled"
-                nzCheckedChildren="On"
-                nzUnCheckedChildren="Off">
-              </nz-switch>
-            </div>
-          </div>
-        </ng-template>
         <div style="margin-left: 5px">
           <nz-badge
             nz-tooltip=""