From a58df35ead6a1529a6e443b993837349d8d2bd1a Mon Sep 17 00:00:00 2001
From: lyzno1 <92089059+lyzno1@users.noreply.github.com>
Date: Fri, 29 Aug 2025 13:37:44 +0800
Subject: [PATCH] fix: improve trigger card layout spacing and remove dividers
 (#24756)

---
 .../components/app/overview/trigger-card.tsx  | 44 ++++++++-----------
 1 file changed, 18 insertions(+), 26 deletions(-)

diff --git a/web/app/components/app/overview/trigger-card.tsx b/web/app/components/app/overview/trigger-card.tsx
index 63ec2567b1..c6a4ad55f4 100644
--- a/web/app/components/app/overview/trigger-card.tsx
+++ b/web/app/components/app/overview/trigger-card.tsx
@@ -4,8 +4,6 @@ import { useTranslation } from 'react-i18next'
 import Link from 'next/link'
 import { Schedule, TriggerAll, WebhookLine } from '@/app/components/base/icons/src/vender/workflow'
 import Switch from '@/app/components/base/switch'
-import Divider from '@/app/components/base/divider'
-import Indicator from '@/app/components/header/indicator'
 import type { AppDetailResponse } from '@/models/app'
 import type { AppSSO } from '@/types/app'
 import { useAppContext } from '@/context/app-context'
@@ -142,32 +140,26 @@ function TriggerCard({ appInfo }: ITriggerCardProps) {
 
         {triggerCount > 0 && (
           <div className="flex flex-col gap-2 p-3">
-            {triggers.map((trigger, index) => (
-              <div key={trigger.id}>
-                <div className="flex w-full items-center gap-3">
-                  <div className="flex grow items-center gap-2">
-                    {getTriggerIcon(trigger)}
-                    <div className="system-sm-medium text-text-secondary">
-                      {trigger.title}
-                    </div>
+            {triggers.map(trigger => (
+              <div key={trigger.id} className="flex w-full items-center gap-3">
+                <div className="flex grow items-center gap-2">
+                  {getTriggerIcon(trigger)}
+                  <div className="system-sm-medium text-text-secondary">
+                    {trigger.title}
                   </div>
-                  <div className="flex items-center gap-1">
-                    <Indicator color={trigger.status === 'enabled' ? 'green' : 'yellow'} />
-                    <div className={`${trigger.status === 'enabled' ? 'text-text-success' : 'text-text-warning'} system-xs-semibold-uppercase`}>
-                      {trigger.status === 'enabled'
-                        ? t('appOverview.overview.status.running')
-                        : t('appOverview.overview.status.disable')}
-                    </div>
-                  </div>
-                  <Switch
-                    defaultValue={trigger.status === 'enabled'}
-                    onChange={enabled => onToggleTrigger(trigger, enabled)}
-                    disabled={!isCurrentWorkspaceEditor}
-                  />
                 </div>
-                {index < triggers.length - 1 && (
-                  <Divider className="my-2" />
-                )}
+                <div className="flex items-center">
+                  <div className={`${trigger.status === 'enabled' ? 'text-text-success' : 'text-text-warning'} system-xs-semibold-uppercase`}>
+                    {trigger.status === 'enabled'
+                      ? t('appOverview.overview.status.running')
+                      : t('appOverview.overview.status.disable')}
+                  </div>
+                </div>
+                <Switch
+                  defaultValue={trigger.status === 'enabled'}
+                  onChange={enabled => onToggleTrigger(trigger, enabled)}
+                  disabled={!isCurrentWorkspaceEditor}
+                />
               </div>
             ))}
           </div>