proposed change of block error message container from P to DIV, to allow better css display property control

stevewasiura · stevewasiura · commit 95ad3464931c · 2012-07-03T11:05:19.000-03:00
diff --git a/jquery.formvalidator.js b/jquery.formvalidator.js
@@ -312,8 +312,8 @@
                     for (var i = 0; i < errorMessages.length; i++) {
                         messages += '<br />* ' + errorMessages[i];
                     }
-
-                    $form.children().eq(0).before('<p class="' + config.errorMessageClass + '">' + messages + '</p>');
+			// using div instead of P gives better control of css display properties
+                    $form.children().eq(0).before('<div class="' + config.errorMessageClass + '">' + messages + '</div>');
                     if(config.scrollToTopOnError) {
                         $(window).scrollTop($form.offset().top - 20);
                     }

Original file line number	Diff line number	Diff line change
`@@ -312,8 +312,8 @@`
`312`	`312`	`for (var i = 0; i < errorMessages.length; i++) {`
`313`	`313`	`messages += '<br />* ' + errorMessages[i];`
`314`	`314`	`}`
`315`		`-`
`316`		`- $form.children().eq(0).before('<p class="' + config.errorMessageClass + '">' + messages + '</p>');`
	`315`	`+ // using div instead of P gives better control of css display properties`
	`316`	`+ $form.children().eq(0).before('<div class="' + config.errorMessageClass + '">' + messages + '</div>');`
`317`	`317`	`if(config.scrollToTopOnError) {`
`318`	`318`	`$(window).scrollTop($form.offset().top - 20);`
`319`	`319`	`}`