Add validHandler support #459

Closed
ericjohannsen opened this Issue Jul 9, 2012 · 4 comments

3 participants

@ericjohannsen

The jQuery validate plugin has an invalidHandler init function which gets triggered if something is invalid. However it does not have anything for validHandler where you you want trigger something immediately if validation succeeds (i.e like hide something).

There is a suggested code change here to resolve this issue:

http://forum.jquery.com/topic/suggestion-for-validate-plugin

The issue is also discussed here:

http://stackoverflow.com/questions/11334365/jquery-event-when-validation-errors-corrected

@ryleyb

submitHandler does that job already, doesn't it? It is triggered when the form is valid.

@ericjohannsen

Based on what I see and the comments on Stack Overflow, it is not triggered when the form again becomes valid but when the form is submitted. The desire is to recognize that input is in a valid state before the form is re-submitted.

@ryleyb

I don't think invalidHandler works the way you think - it only triggers on form submit also.

Perhaps you could do what you want by using the highlight and unhighlight options?

@jzaefferer
Owner

I'm sorry for the lack of activity on this issue. Instead of leaving it open any longer, I decided to close old issues without trying to address them, to longer give the false impression that it will get addressed eventually.

To the reporter (or anyone else interested in this issue): If you're affected by the same issue, consider opening a new issue, with a testpage that shows the issue. Even better, try to fix the issue yourself, and improve the project by sending a pull request. This may seem daunting at first, but you'll likely learn some useful skills that you can apply elsewhere as well. And you can help keep this project alive. We've documented how to do these things, too. A patch is worth a thousand issues!

@jzaefferer jzaefferer closed this Jun 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment