Merge pull request jquery-validation#1628 from alokito/pendingClass2

Adds pending class to elements while remote operations

src/core.js

@@ -225,6 +225,7 @@ $.extend( $.validator, {
 		groups: {},
 		rules: {},
 		errorClass: "error",
+		pendingClass: "pending",
 		validClass: "valid",
 		errorElement: "label",
 		focusCleanup: false,
@@ -938,6 +939,7 @@ $.extend( $.validator, {
 		startRequest: function( element ) {
 			if ( !this.pending[ element.name ] ) {
 				this.pendingRequest++;
+				$( element ).addClass( this.settings.pendingClass );
 				this.pending[ element.name ] = true;
 			}
 		},
@@ -949,6 +951,7 @@ $.extend( $.validator, {
 				this.pendingRequest = 0;
 			}
 			delete this.pending[ element.name ];
+			$( element ).removeClass( this.settings.pendingClass );
 			if ( valid && this.pendingRequest === 0 && this.formSubmitted && this.form() ) {
 				$( this.currentForm ).submit();
 				this.formSubmitted = false;

test/methods.js

@@ -396,6 +396,29 @@ asyncTest( "remote", function() {
 	strictEqual( v.element( e ), true, "still invalid, because remote validation must block until it returns; dependency-mismatch considered as valid though" );
 } );
 
+asyncTest( "remote, pending class added to element while call outstanding", function() {
+	expect( 3 );
+	var e = $( "#username" ),
+		v = $( "#userForm" ).validate( {
+			rules: {
+				username: {
+					remote: {
+						url: "users.php",
+						complete: function() {
+							strictEqual( e.hasClass( "pending" ), false, "not pending since ajax call complete" );
+							start();
+						}
+					}
+				}
+			}
+		} );
+	strictEqual( e.hasClass( "pending" ), false, "not pending since no data entered" );
+	e.val( "Peter" );
+	// this fires off the validation:
+	v.element( e );
+	strictEqual( e.hasClass( "pending" ), true, "pending while validation outstanding" );
+} );
+
 asyncTest( "remote, customized ajax options", function() {
 	expect( 2 );
 	$( "#userForm" ).validate( {