netteForms: alert() replaced with <span class="nette-form-error">

[dg]

No description provided.

[Majkl578]

👎
Even though I'm not fan of alerts, I don't like this even more. I think current behavior is pragmatic and should stay as is - not affecting DOM in any way. If one ones to change it, there is no problem with that (we already have few live validators etc.).

[fprochazka]

I like it. It should be as an alternative, that you can turn on by something like

Nette.addError = Nette.showFormErrors;

This is the intended behaviour, right?

---

Oh, where went addError ?

[enumag]

IMO the JS validation should be triggered after change / blur of any field in the form (not just when the user tries to submit the form) and display the errors in the same places as server-side validation - that is next to the inputs. I've already implemented it (using @hosiplan's twitter bootstrap renderer) and it is very convenient for the user.
EDIT: You can find my implementation here: https://gist.github.com/enumag/5687497.
I'm not sure if such behaviour should be default but I like it.

[smasty]

How about using setCustomValidity on the elements if supported by the browser?

[dg]

@enumag now you can use default Nette.validateForm.

[dg]

Moved to nette/forms#9