averello/jquery-validation-mirror
1
0
Fork 0
mirror of https://github.com/jquery-validation/jquery-validation.git synced 2025-12-20 09:11:40 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fixed handling of multiple class names for errorClass option. Test by Max Lynch. Fixes #280

Browse Source
This commit is contained in:
Başar Aykut
2011-12-09 14:04:45 +02:00
committed by Jörn Zaefferer
parent d8210bf098
commit bafc78671a
2 changed files with 17 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
jquery.validate.js vendored
View File

@@ -490,7 +490,8 @@ $.extend($.validator, {
},
errors: function() {
return $( this.settings.errorElement + "." + this.settings.errorClass, this.errorContext );
var errorClass = this.settings.errorClass.replace(' ', '.');
return $( this.settings.errorElement + "." + errorClass, this.errorContext );
},
reset: function() {

15
test/test.js
View File

@@ -445,6 +445,21 @@ test("option: focusCleanup with wrapper", function() {
ok( !form.is(":has(span:visible:has(label.error[for=username]))") );
});
test("option: errorClass with multiple classes", function() {
var form = $("#userForm")
form.validate({
focusCleanup: true,
wrapper: "span",
errorClass: "error error1"
});
form.valid();
ok( form.is(":has(span:visible:has(label.error[for=username]))") );
ok( form.is(":has(span:visible:has(label.error1[for=username]))") );
$("#username").focus().trigger("focusin");
ok( !form.is(":has(span:visible:has(label.error[for=username]))") );
ok( !form.is(":has(span:visible:has(label.error1[for=username]))") );
});
test("elements() order", function() {
var container = $("#orderContainer");
var v = $("#elementsOrder").validate({