From 9ed45594c6f85218f6a5bf62143862dd90b9f776 Mon Sep 17 00:00:00 2001
From: lyzno1 <92089059+lyzno1@users.noreply.github.com>
Date: Sun, 31 Aug 2025 16:59:49 +0800
Subject: [PATCH] fix: improve schedule trigger and quick settings
 app-operation btns ui (#24843)

---
 .../components/app-sidebar/app-operations.tsx |  4 ++--
 .../components/execute-now-button.tsx         | 24 -------------------
 .../components/next-execution-times.tsx       | 10 ++++----
 .../workflow/nodes/trigger-schedule/panel.tsx |  9 -------
 4 files changed, 8 insertions(+), 39 deletions(-)
 delete mode 100644 web/app/components/workflow/nodes/trigger-schedule/components/execute-now-button.tsx

diff --git a/web/app/components/app-sidebar/app-operations.tsx b/web/app/components/app-sidebar/app-operations.tsx
index 2b6f33cdb1..bbd580b065 100644
--- a/web/app/components/app-sidebar/app-operations.tsx
+++ b/web/app/components/app-sidebar/app-operations.tsx
@@ -62,7 +62,7 @@ const AppOperations = ({ primaryOperations, secondaryOperations, gap }: {
           key={operation.id}
           size={'small'}
           variant={'secondary'}
-          className="gap-[1px]"
+          className="gap-[1px] px-1.5"
           onClick={operation.onClick}>
           {cloneElement(operation.icon, { className: 'h-3.5 w-3.5 text-components-button-secondary-text' })}
           <span className="system-xs-medium text-components-button-secondary-text">
@@ -85,7 +85,7 @@ const AppOperations = ({ primaryOperations, secondaryOperations, gap }: {
             <Button
               size={'small'}
               variant={'secondary'}
-              className='gap-1'
+              className='gap-1 px-1.5'
             >
               <RiMoreFill className='h-3.5 w-3.5 text-components-button-secondary-text' />
               <span className='system-xs-medium text-components-button-secondary-text'>{t('common.operation.more')}</span>
diff --git a/web/app/components/workflow/nodes/trigger-schedule/components/execute-now-button.tsx b/web/app/components/workflow/nodes/trigger-schedule/components/execute-now-button.tsx
deleted file mode 100644
index 8acf252c05..0000000000
--- a/web/app/components/workflow/nodes/trigger-schedule/components/execute-now-button.tsx
+++ /dev/null
@@ -1,24 +0,0 @@
-import React from 'react'
-import { useTranslation } from 'react-i18next'
-
-type ExecuteNowButtonProps = {
-  onClick: () => void
-  disabled?: boolean
-}
-
-const ExecuteNowButton = ({ onClick, disabled = false }: ExecuteNowButtonProps) => {
-  const { t } = useTranslation()
-
-  return (
-    <button
-      type="button"
-      onClick={onClick}
-      disabled={disabled}
-      className="w-full rounded-lg border-[0.5px] border-components-button-secondary-border bg-components-button-secondary-bg py-1.5 text-xs font-medium text-components-button-secondary-text shadow-xs hover:bg-components-button-secondary-bg-hover disabled:cursor-not-allowed disabled:opacity-50"
-    >
-      {t('workflow.nodes.triggerSchedule.executeNow')}
-    </button>
-  )
-}
-
-export default ExecuteNowButton
diff --git a/web/app/components/workflow/nodes/trigger-schedule/components/next-execution-times.tsx b/web/app/components/workflow/nodes/trigger-schedule/components/next-execution-times.tsx
index 063fc95adf..02e85e2724 100644
--- a/web/app/components/workflow/nodes/trigger-schedule/components/next-execution-times.tsx
+++ b/web/app/components/workflow/nodes/trigger-schedule/components/next-execution-times.tsx
@@ -23,13 +23,15 @@ const NextExecutionTimes = ({ data }: NextExecutionTimesProps) => {
       <label className="block text-xs font-medium text-gray-500">
         {t('workflow.nodes.triggerSchedule.nextExecutionTimes')}
       </label>
-      <div className="space-y-2 rounded-lg bg-components-input-bg-normal p-3">
+      <div className="flex min-h-[80px] flex-col rounded-xl bg-components-input-bg-normal py-2">
         {executionTimes.map((time, index) => (
-          <div key={index} className="flex items-baseline gap-3 text-xs text-text-secondary">
-            <span className="select-none font-mono leading-none text-text-quaternary">
+          <div key={index} className="flex items-baseline text-xs">
+            <span className="w-6 select-none text-right font-mono font-normal leading-[150%] tracking-wider text-text-quaternary">
               {String(index + 1).padStart(2, '0')}
             </span>
-            <span className="leading-none">{time}</span>
+            <span className="pl-2 pr-3 font-mono font-normal leading-[150%] tracking-wider text-text-secondary">
+              {time}
+            </span>
           </div>
         ))}
       </div>
diff --git a/web/app/components/workflow/nodes/trigger-schedule/panel.tsx b/web/app/components/workflow/nodes/trigger-schedule/panel.tsx
index 2ae352ce21..743addcaec 100644
--- a/web/app/components/workflow/nodes/trigger-schedule/panel.tsx
+++ b/web/app/components/workflow/nodes/trigger-schedule/panel.tsx
@@ -10,7 +10,6 @@ import WeekdaySelector from './components/weekday-selector'
 import TimePicker from '@/app/components/base/date-and-time-picker/time-picker'
 import dayjs from 'dayjs'
 import NextExecutionTimes from './components/next-execution-times'
-import ExecuteNowButton from './components/execute-now-button'
 import MonthlyDaysSelector from './components/monthly-days-selector'
 import OnMinuteSelector from './components/on-minute-selector'
 import Input from '@/app/components/base/input'
@@ -34,11 +33,6 @@ const Panel: FC<NodePanelProps<ScheduleTriggerNodeType>> = ({
     handleOnMinuteChange,
   } = useConfig(id, data)
 
-  const handleExecuteNow = () => {
-    // TODO: Implement execute now functionality
-    console.log('Execute now clicked')
-  }
-
   return (
     <div className='mt-2'>
       <div className='space-y-4 px-4 pb-3 pt-2'>
@@ -146,9 +140,6 @@ const Panel: FC<NodePanelProps<ScheduleTriggerNodeType>> = ({
 
         <NextExecutionTimes data={inputs} />
 
-        <div className="pt-2">
-          <ExecuteNowButton onClick={handleExecuteNow} />
-        </div>
       </div>
     </div>
   )