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.

Sign up for GitHub

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

Jump to bottom

Update Form.Validator.Extras.js #1186

Closed
wants to merge 1 commit into from
Closed

Update Form.Validator.Extras.js #1186

wants to merge 1 commit into from

Conversation

kelexel
Copy link
Contributor

@kelexel kelexel commented Mar 5, 2013

Fix validate-match "matchName" not "working" if containing spaces
(See #1185 )
@kelexel
Update Form.Validator.Extras.js
f78a36c 
Fix validate-match "matchName" not "working" if containing spaces
@Shic1983
Copy link
Contributor

Shic1983 commented Mar 7, 2013

Looks good, hopefully this will be merged soon..
@SergioCrisostomo
Copy link
Member

LGTM. Specs passed.
@anutron @arian ?
@anutron
Copy link
Member

anutron commented Apr 27, 2014

there are no specs for fv.extras. can we get a jsfiddle demonstrating the issue and the fix?
@SergioCrisostomo
Copy link
Member

@kelexel @NTOM could you please make a jsFiddle reproducing the problem here? Thank you!
@SergioCrisostomo
Copy link
Member

Related: #1213
@anutron
Copy link
Member

anutron commented Oct 27, 2015

I guess no jsFiddle. @SergioCrisostomo I'm ok with this shipping. It looks good to me.
@SergioCrisostomo
Copy link
Member

merged in 4a6cabe
@mootools mootools deleted a comment Mar 25, 2022
@mootools mootools deleted a comment Mar 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants
@kelexel @Shic1983 @SergioCrisostomo @anutron