From d8ddbc4d87e57ee7baa37a960d57373081cc6722 Mon Sep 17 00:00:00 2001
From: lyzno1 <92089059+lyzno1@users.noreply.github.com>
Date: Fri, 29 Aug 2025 14:24:23 +0800
Subject: [PATCH] feat: enhance webhook trigger panel UI consistency and
 interactivity (#24759)

Co-authored-by: hjlarry <hjlarry@163.com>
---
 web/app/components/base/select/index.tsx      |  2 +-
 .../workflow/nodes/trigger-webhook/panel.tsx  | 68 +++++++++++++++----
 web/i18n/en-US/workflow.ts                    |  3 +
 web/i18n/ja-JP/workflow.ts                    |  3 +
 web/i18n/zh-Hans/workflow.ts                  |  3 +
 5 files changed, 65 insertions(+), 14 deletions(-)

diff --git a/web/app/components/base/select/index.tsx b/web/app/components/base/select/index.tsx
index 9ad2a489b0..fe082b31af 100644
--- a/web/app/components/base/select/index.tsx
+++ b/web/app/components/base/select/index.tsx
@@ -286,7 +286,7 @@ const SimpleSelect: FC<ISelectProps> = ({
                             {selected && !hideChecked && (
                               <span
                                 className={classNames(
-                                  'absolute inset-y-0 right-0 flex items-center pr-4 text-text-accent',
+                                  'absolute inset-y-0 right-0 flex items-center pr-2 text-text-accent',
                                 )}
                               >
                                 <RiCheckLine className="h-4 w-4" aria-hidden="true" />
diff --git a/web/app/components/workflow/nodes/trigger-webhook/panel.tsx b/web/app/components/workflow/nodes/trigger-webhook/panel.tsx
index 5c58a0000c..f7a759f5f5 100644
--- a/web/app/components/workflow/nodes/trigger-webhook/panel.tsx
+++ b/web/app/components/workflow/nodes/trigger-webhook/panel.tsx
@@ -11,9 +11,11 @@ import Split from '@/app/components/workflow/nodes/_base/components/split'
 import type { NodePanelProps } from '@/app/components/workflow/types'
 import InputWithCopy from '@/app/components/base/input-with-copy'
 import Input from '@/app/components/base/input'
-import Select from '@/app/components/base/select'
+import { SimpleSelect } from '@/app/components/base/select'
 import Switch from '@/app/components/base/switch'
 import Toast from '@/app/components/base/toast'
+import Tooltip from '@/app/components/base/tooltip'
+import copy from 'copy-to-clipboard'
 
 const i18nPrefix = 'workflow.nodes.triggerWebhook'
 
@@ -39,6 +41,7 @@ const Panel: FC<NodePanelProps<WebhookTriggerNodeType>> = ({
   data,
 }) => {
   const { t } = useTranslation()
+  const [debugUrlCopied, setDebugUrlCopied] = React.useState(false)
   const {
     readOnly,
     inputs,
@@ -67,18 +70,22 @@ const Panel: FC<NodePanelProps<WebhookTriggerNodeType>> = ({
       <div className='space-y-4 px-4 pb-3 pt-2'>
         {/* Webhook URL Section */}
         <Field title={t(`${i18nPrefix}.webhookUrl`)}>
-          <div className="space-y-2">
-            <div className="flex gap-2">
-              <div className="w-28 shrink-0">
-                <Select
+          <div className="space-y-1">
+            <div className="flex gap-1" style={{ width: '368px', height: '32px' }}>
+              <div className="w-26 shrink-0">
+                <SimpleSelect
                   items={HTTP_METHODS}
                   defaultValue={inputs.method}
                   onSelect={item => handleMethodChange(item.value as HttpMethod)}
                   disabled={readOnly}
+                  className="h-8 pr-8 text-sm"
+                  wrapperClassName="h-8"
+                  optionWrapClassName="w-26 min-w-26 z-[5]"
                   allowSearch={false}
+                  notClearable={true}
                 />
               </div>
-              <div className="flex flex-1 gap-2">
+              <div className="flex-1" style={{ width: '284px' }}>
                 <InputWithCopy
                   value={inputs.webhook_url || ''}
                   placeholder={t(`${i18nPrefix}.webhookUrlPlaceholder`)}
@@ -92,6 +99,35 @@ const Panel: FC<NodePanelProps<WebhookTriggerNodeType>> = ({
                 />
               </div>
             </div>
+            {inputs.webhook_debug_url && (
+              <Tooltip
+                popupContent={debugUrlCopied ? t(`${i18nPrefix}.debugUrlCopied`) : t(`${i18nPrefix}.debugUrlCopy`)}
+                popupClassName="system-xs-regular text-text-primary bg-components-tooltip-bg border border-components-panel-border shadow-lg backdrop-blur-sm rounded-md px-1.5 py-1"
+                position="top"
+                offset={{ mainAxis: -20 }}
+                needsDelay={false}
+              >
+                <div
+                  className="flex cursor-pointer gap-1.5 rounded-lg px-1 py-1.5 transition-colors"
+                  style={{ width: '368px', height: '38px' }}
+                  onClick={() => {
+                    copy(inputs.webhook_debug_url || '')
+                    setDebugUrlCopied(true)
+                    setTimeout(() => setDebugUrlCopied(false), 2000)
+                  }}
+                >
+                  <div className="mt-0.5 w-0.5 bg-divider-regular" style={{ height: '28px' }}></div>
+                  <div className="flex-1" style={{ width: '352px', height: '32px' }}>
+                    <div className="text-xs leading-4 text-text-tertiary">
+                      {t(`${i18nPrefix}.debugUrlTitle`)}
+                    </div>
+                    <div className="truncate text-xs leading-4 text-text-primary">
+                      {inputs.webhook_debug_url}
+                    </div>
+                  </div>
+                </div>
+              </Tooltip>
+            )}
           </div>
         </Field>
         <span>{inputs.webhook_debug_url || ''}</span>
@@ -100,13 +136,19 @@ const Panel: FC<NodePanelProps<WebhookTriggerNodeType>> = ({
 
         {/* Content Type */}
         <Field title={t(`${i18nPrefix}.contentType`)}>
-          <Select
-            items={CONTENT_TYPES}
-            defaultValue={inputs['content-type']}
-            onSelect={item => handleContentTypeChange(item.value as string)}
-            disabled={readOnly}
-            allowSearch={false}
-          />
+          <div className="w-full">
+            <SimpleSelect
+              items={CONTENT_TYPES}
+              defaultValue={inputs['content-type']}
+              onSelect={item => handleContentTypeChange(item.value as string)}
+              disabled={readOnly}
+              className="h-8 text-sm"
+              wrapperClassName="h-8"
+              optionWrapClassName="min-w-48 z-[5]"
+              allowSearch={false}
+              notClearable={true}
+            />
+          </div>
         </Field>
 
         <Split />
diff --git a/web/i18n/en-US/workflow.ts b/web/i18n/en-US/workflow.ts
index 5163453268..a3dc6f0003 100644
--- a/web/i18n/en-US/workflow.ts
+++ b/web/i18n/en-US/workflow.ts
@@ -1017,6 +1017,9 @@ const translation = {
       noQueryParameters: 'No query parameters configured',
       noHeaders: 'No headers configured',
       noBodyParameters: 'No body parameters configured',
+      debugUrlTitle: 'For test runs, always use this URL',
+      debugUrlCopy: 'Click to copy',
+      debugUrlCopied: 'Copied!',
       errorHandling: 'Error Handling',
       errorStrategy: 'Error Handling',
       responseConfiguration: 'Response Configuration',
diff --git a/web/i18n/ja-JP/workflow.ts b/web/i18n/ja-JP/workflow.ts
index 1690c88074..ce91b6fde9 100644
--- a/web/i18n/ja-JP/workflow.ts
+++ b/web/i18n/ja-JP/workflow.ts
@@ -1017,6 +1017,9 @@ const translation = {
       noQueryParameters: 'クエリパラメータは設定されていません',
       noHeaders: 'ヘッダーは設定されていません',
       noBodyParameters: 'ボディパラメータは設定されていません',
+      debugUrlTitle: 'テスト実行には、常にこのURLを使用してください',
+      debugUrlCopy: 'クリックしてコピー',
+      debugUrlCopied: 'コピーしました！',
       errorHandling: 'エラー処理',
       errorStrategy: 'エラー処理',
       responseConfiguration: 'レスポンス設定',
diff --git a/web/i18n/zh-Hans/workflow.ts b/web/i18n/zh-Hans/workflow.ts
index 390d01f39c..e497ec7f9b 100644
--- a/web/i18n/zh-Hans/workflow.ts
+++ b/web/i18n/zh-Hans/workflow.ts
@@ -1017,6 +1017,9 @@ const translation = {
       noQueryParameters: '未配置查询参数',
       noHeaders: '未配置 Header',
       noBodyParameters: '未配置请求体参数',
+      debugUrlTitle: '测试运行时，请始终使用此URL',
+      debugUrlCopy: '点击复制',
+      debugUrlCopied: '已复制！',
       errorHandling: '错误处理',
       errorStrategy: '错误处理',
       responseConfiguration: '响应配置',