Skip to content
This repository has been archived by the owner on Nov 10, 2018. It is now read-only.

Update validate.js #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update validate.js #47

wants to merge 1 commit into from

Conversation

tbb2
Copy link

@tbb2 tbb2 commented Apr 10, 2018

If you make your own validation (e.g. I had to compare the values of 2 fields) using
field.setCustomValidity(customErrorMessageOfSomeKind)
the generic error will currently be shown, just have to add these 3 lines to catch the right error message (I apologize for being lazy and just changing this one file and not in the dist folder etc.)

If you make your own validation (e.g. I had to compare the values of 2 fields) using 
field.setCustomValidity(customErrorMessageOfSomeKind)
the generic error will currently be shown, just have to add these 3 lines to catch the right error message (I apologize for being lazy and just changing this one file and not in the dist folder etc.)
@cferdinandi
Copy link
Owner

Possible duplicate of #46?

@tbb2
Copy link
Author

tbb2 commented Apr 12, 2018

Maybe 2 sides of the same issue? #46 is just about the polyfill, but there's apparently also no support for it without the polyfill

@cferdinandi
Copy link
Owner

Sorry for the delay. I'll merge this soon!

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

Successfully merging this pull request may close these issues.

2 participants