Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Key up validation #26

Closed
rhenline opened this issue Jan 25, 2013 · 3 comments
Closed

Key up validation #26

rhenline opened this issue Jan 25, 2013 · 3 comments
Assignees

Comments

@rhenline
Copy link

Would it not be better to do a key up for validation rather than on Focus? I have seen many other validation plugins that use this method. Right now fields do not validate until the focus is changed. So, someone might get confused and not click the submit button or move to the next field because the current field is still in red and they don't know that they need to change the focus by clicking on something or moving to the next field.

@ghost ghost assigned uzquiano Jan 31, 2013
@uzquiano
Copy link
Member

Very good points. Makes sense to switch the event to keyUp. If you have a PR, I'd be happy to apply it otherwise, I should have some time over the next two days to work on this.

@uzquiano
Copy link
Member

uzquiano commented Feb 4, 2013

I just committed a change where keyUp validates field control state. Validation also occurs for blur (as was default behavior). The benefit, as pointed out by @rhenline, is that the user now gets a better sense of when they've input a good or a bad value as the feedback is immediate.

@uzquiano uzquiano closed this as completed Feb 4, 2013
@ip6li
Copy link

ip6li commented Oct 12, 2015

How can "Validate on KeyUp" be activated?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants