Constraint validation for two-way binding

[Superpat]

Just throwing this out here to see if it's a good idea before I commit to building out the go example page and playground test:

If an html input has any constraints on it, then only update the signal if it's valid.

index 3bd0754..497b9bb 100644
--- a/packages/library/src/lib/plugins/attributes.ts
+++ b/packages/library/src/lib/plugins/attributes.ts
@@ -143,6 +143,8 @@ export const TwoWayBindingModelPlugin: AttributePlugin = {
       const current = signal.value
       const input = el as HTMLInputElement
 
+      if (!input.validity.valid) return
+
       if (typeof current === 'number') {
         signal.value = Number(input.value)
       } else if (typeof current === 'string') {

Seems to me like it would reduce most of the needs for custom validation code in simple forms and make it all
more declarative.

I have an idea on how to add validation on signal to input changes too, but unsure if it's desirable.

[Superpat]

@delaneyj it's not duplicating the validation, that's the browser api for checking if an element is valid. It's supposed to be much faster then js based validation: https://developer.mozilla.org/en-US/docs/Web/HTML/Constraint_validation

[delaneyj]

Ah, sorry I miss understood. I think it might need a debounce but it's a good idea!

[Superpat]

Is the whole on input listener debounced ?

[Superpat]

I will get to this after EuroBSDCon

[bencroker]

I’ve created #293 for further discussion.