chenlica opened a new pull request, #5118: URL: https://github.com/apache/texera/pull/5118
### What changes were proposed in this PR? Remove the "Execution Settings" hover popover that appears next to the Run button in the workflow editor. The popover contained a custom-execution-name input and an email-notification toggle, but the same defaults work for ~all users and the hover trigger was felt to be unnecessary UX clutter. - Drop the `nz-popover` directives and the `<ng-template #executionSettings>` block from `menu.component.html`. - The `runWorkflow()` handler already falls back to `"Untitled Execution"` when `currentExecutionName` is empty, so behavior on click is unchanged. - `currentExecutionName` (still declared as an unused `@Input`) and the email-notification toggle config flag remain in place for now; cleaning them up can be a follow-up if there's interest. ### Any related issues, documentation, discussions? No linked issue — small UX cleanup. ### How was this PR tested? Verified locally by applying the same edit to a running `ng serve` dev server: the hot-reloaded bundle no longer contains the `"Execution Settings"` string, and the Run button still triggers `runWorkflow()` with the default `"Untitled Execution"` name on click. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Claude Opus 4.7) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
