Change TextInput to only have onChange(string) for better UX

Author: mythz

src/components/TextInput.tsx

@@ -83,7 +83,8 @@ const TextInput = forwardRef<TextInputRef, TextInputProps & Omit<React.InputHTML
     const handleInput = (e: React.ChangeEvent<HTMLInputElement>) => {
       if (onChange) {
         const newValue = textInputValue(useType, e.target.value)
-        onChange(newValue)
+        // Improve UX by always returning a string for text inputs
+        onChange(typeof newValue == 'string' ? newValue : `${newValue ?? ''}`)
       }
     }
 

src/components/types.ts

@@ -21,7 +21,7 @@ export interface TextInputProps {
   help?: string
   placeholder?: string
   value?: string|number
-  onChange?: (value:string|number) => void
+  onChange?: (value:string) => void
 }
 export interface TextInputRef {
   focus(): void