fix: update click handler logic in OptionCard

pull/21398/head
twwu 1 year ago
parent f481075f8f
commit 8d4ced227e

@ -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 }) => { const ParamItem: FC<Props> = ({ className, id, name, noTooltip, tip, step = 0.1, min = 0, max, value, enable, onChange, hasSwitch, onSwitchChange }) => {
return ( return (
<div className={className}> <div className={className}>
<div className="flex items-center justify-between"> <div className='flex items-center justify-between'>
<div className="flex h-6 items-center"> <div className='flex h-6 items-center'>
{hasSwitch && ( {hasSwitch && (
<Switch <Switch
size='md' 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 && ( {!noTooltip && (
<Tooltip <Tooltip
triggerClassName='w-4 h-4 shrink-0' triggerClassName='w-4 h-4 shrink-0'
popupContent={<div className="w-[200px]">{tip}</div>} popupContent={<div className='w-[200px]'>{tip}</div>}
/> />
)} )}
</div> </div>
</div> </div>
<div className="mt-1 flex items-center"> <div className='mt-1 flex items-center'>
<div className="mr-3 flex shrink-0 items-center"> <div className='mr-3 flex shrink-0 items-center'>
<InputNumber <InputNumber
disabled={!enable} disabled={!enable}
type="number" type='number'
min={min} min={min}
max={max} max={max}
step={step} step={step}
@ -62,7 +62,7 @@ const ParamItem: FC<Props> = ({ className, id, name, noTooltip, tip, step = 0.1,
className='w-[72px]' className='w-[72px]'
/> />
</div> </div>
<div className="flex grow items-center"> <div className='flex grow items-center'>
<Slider <Slider
className='w-full' className='w-full'
disabled={!enable} disabled={!enable}

@ -57,7 +57,7 @@ const OptionCard = <T,>({
disabled && 'cursor-not-allowed opacity-50', disabled && 'cursor-not-allowed opacity-50',
)} )}
onClick={() => { onClick={() => {
if (disabled) return if (isActive || disabled) return
onClick?.(id) onClick?.(id)
}} }}
> >

Loading…
Cancel
Save