opensource/sim
3
0
Fork 0
mirror of https://github.com/simstudioai/sim.git synced 2026-09-24 15:45:35 +08:00
Code Issues Packages Projects Releases Wiki Activity

Fix: range input experience

Browse Source
This commit is contained in:
Emir Karabeg
2025-02-12 19:37:02 -08:00
parent a4cf9bf8cc
commit e2a5e39b0b
1 changed files with 24 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/w/[id]/components/workflow-block/components/sub-block/components/eval-input.tsx
+24 -7
View File
@@ -75,8 +75,25 @@ export function EvalInput({ blockId, subBlockId }: EvalInputProps) {
}
const updateRange = (id: string, field: 'min' | 'max', value: string) => {
const numValue = parseFloat(value)
if (isNaN(numValue)) return
setValue(
metrics.map((metric) =>
metric.id === id
? {
...metric,
range: {
...metric.range,
[field]: value,
},
}
: metric
)
)
}
const handleRangeBlur = (id: string, field: 'min' | 'max', value: string) => {
// Allow any number including negatives, just remove non-numeric characters except - and .
const sanitizedValue = value.replace(/[^\d.-]/g, '')
const numValue = parseFloat(sanitizedValue)
setValue(
metrics.map((metric) =>
@@ -85,7 +102,7 @@ export function EvalInput({ blockId, subBlockId }: EvalInputProps) {
...metric,
range: {
...metric.range,
[field]: numValue,
[field]: !isNaN(numValue) ? numValue : 0,
},
}
: metric
@@ -206,20 +223,20 @@ export function EvalInput({ blockId, subBlockId }: EvalInputProps) {
<div className="space-y-1">
<Label>Min Value</Label>
<Input
type="number"
type="text"
value={metric.range.min}
onChange={(e) => updateRange(metric.id, 'min', e.target.value)}
step={0.1}
onBlur={(e) => handleRangeBlur(metric.id, 'min', e.target.value)}
className="text-muted-foreground placeholder:text-muted-foreground/50"
/>
</div>
<div className="space-y-1">
<Label>Max Value</Label>
<Input
type="number"
type="text"
value={metric.range.max}
onChange={(e) => updateRange(metric.id, 'max', e.target.value)}
step={0.1}
onBlur={(e) => handleRangeBlur(metric.id, 'max', e.target.value)}
className="text-muted-foreground placeholder:text-muted-foreground/50"
/>
</div>