svelte-forms

svelte-forms copied to clipboard

Describe the bug

The function min(5) doesn't work if the value is a string of numbers ONLY ie: '123'.

To Reproduce Steps to reproduce the behavior:

1. use min(5) with input '123'
2. the output will be valid:true

Expected behavior the output on the above example should be valid: false

Solution

change: const val = isNaN(value) ? value.length : parseFloat(value); to: const val = typeof value === 'string' ? value.length : isNaN(value) ? 0 : parseFloat(value)

I just saw multiple issues for this. https://github.com/chainlist/svelte-forms/issues/103, https://github.com/chainlist/svelte-forms/issues/88

I have a proposed solution in the description above