From 8d4ced227ee4a215587c403d1179f07e3707f5df Mon Sep 17 00:00:00 2001
From: twwu <twwu@dify.ai>
Date: Fri, 16 May 2025 17:54:41 +0800
Subject: [PATCH] fix: update click handler logic in OptionCard

---
 web/app/components/base/param-item/index.tsx     | 16 ++++++++--------
 .../common/retrieval-param-config/index.tsx      |  2 +-
 .../components/datasets/settings/option-card.tsx |  4 ++--
 3 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/web/app/components/base/param-item/index.tsx b/web/app/components/base/param-item/index.tsx
index 03eb5a7c42..ab6d4bf20f 100644
--- a/web/app/components/base/param-item/index.tsx
+++ b/web/app/components/base/param-item/index.tsx
@@ -24,8 +24,8 @@ type Props = {
 const ParamItem: FC<Props> = ({ className, id, name, noTooltip, tip, step = 0.1, min = 0, max, value, enable, onChange, hasSwitch, onSwitchChange }) => {
   return (
     <div className={className}>
-      <div className="flex items-center justify-between">
-        <div className="flex h-6 items-center">
+      <div className='flex items-center justify-between'>
+        <div className='flex h-6 items-center'>
           {hasSwitch && (
             <Switch
               size='md'
@@ -36,20 +36,20 @@ const ParamItem: FC<Props> = ({ className, id, name, noTooltip, tip, step = 0.1,
               }}
             />
           )}
-          <span className="system-sm-semibold mr-1 text-text-secondary">{name}</span>
+          <span className='system-sm-semibold mr-1 text-text-secondary'>{name}</span>
           {!noTooltip && (
             <Tooltip
               triggerClassName='w-4 h-4 shrink-0'
-              popupContent={<div className="w-[200px]">{tip}</div>}
+              popupContent={<div className='w-[200px]'>{tip}</div>}
             />
           )}
         </div>
       </div>
-      <div className="mt-1 flex items-center">
-        <div className="mr-3 flex shrink-0 items-center">
+      <div className='mt-1 flex items-center'>
+        <div className='mr-3 flex shrink-0 items-center'>
           <InputNumber
             disabled={!enable}
-            type="number"
+            type='number'
             min={min}
             max={max}
             step={step}
@@ -62,7 +62,7 @@ const ParamItem: FC<Props> = ({ className, id, name, noTooltip, tip, step = 0.1,
             className='w-[72px]'
           />
         </div>
-        <div className="flex grow items-center">
+        <div className='flex grow items-center'>
           <Slider
             className='w-full'
             disabled={!enable}
diff --git a/web/app/components/datasets/common/retrieval-param-config/index.tsx b/web/app/components/datasets/common/retrieval-param-config/index.tsx
index 97e0a7f303..2ad5317faa 100644
--- a/web/app/components/datasets/common/retrieval-param-config/index.tsx
+++ b/web/app/components/datasets/common/retrieval-param-config/index.tsx
@@ -202,7 +202,7 @@ const RetrievalParamConfig: FC<Props> = ({
                       option.value === RerankingModeEnum.WeightedScore
                         ? ProgressIndicator
                         : Reranking
-                    } alt=''/>}
+                    } alt='' />}
                     title={option.label}
                     description={option.tips}
                     className='flex-1'
diff --git a/web/app/components/datasets/settings/option-card.tsx b/web/app/components/datasets/settings/option-card.tsx
index 15e69db2bc..d25ba90061 100644
--- a/web/app/components/datasets/settings/option-card.tsx
+++ b/web/app/components/datasets/settings/option-card.tsx
@@ -57,7 +57,7 @@ const OptionCard = <T,>({
         disabled && 'cursor-not-allowed opacity-50',
       )}
       onClick={() => {
-        if (disabled) return
+        if (isActive || disabled) return
         onClick?.(id)
       }}
     >
@@ -70,7 +70,7 @@ const OptionCard = <T,>({
             <div className={cn(
               'absolute left-[-2px] top-[-2px] h-14 w-14 rounded-full blur-[80px]',
               `${HEADER_EFFECT_MAP[effectColor]}`,
-            )}/>
+            )} />
           )
         }
         {