Conservancy/website
6
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Rework Javascript final form validation.

Browse source 
The problem before was that an error in the annual form would prevent
submission of the monthly form and vice-versa.  That is herein corrected
with this change, which assures that the input with id of "amount" if
the specific form (id'd with "annual" or "monthly") is the only one
checked.
This commit is contained in:
Bradley M. Kuhn 2014-12-03 20:44:15 -05:00
parent 9c40bf2e60
commit 2d96daee90
1 changed files with 14 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
www/conservancy/static/supporter-page.js
View file

@ -64,8 +64,8 @@ $(document).ready(function() {
errorElement.removeClass("form-error").addClass("form-error-show");
}
});
$("#supporter-form-submit").click(function(event){
var valid = $('.amount').hasClass("valid");
var validateFormAtSubmission = function(element, event) {
var valid = element.hasClass("valid");
if (! valid) {
$("#form-correction-needed").removeClass("form-error").addClass("form-error-show")
.css("font-weight", "bold").css("font-size", "150%");
@ -73,6 +73,12 @@ $(document).ready(function() {
} else {
$("#form-correction-needed").removeClass("form-error-show").addClass("form-error");
}
};
$(".supporter-form-submit#monthly").click(function (event) {
validateFormAtSubmission($(".supporter-form#monthly input#amount"), event);
});
$(".supporter-form-submit#annual").click(function (event) {
validateFormAtSubmission($(".supporter-form#annual input#amount"), event);
});
/* Handle toggling of annual/monthly form selections */
$('.supporter-type-selection#monthly').hide();