tonyair
/
silverstripe-comments
mirror of https://github.com/silverstripe/silverstripe-comments
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Added JS fix for reply forms - fixes issue where the submit button breaks due to incorrect use of the jQuery validate function

This commit is contained in:
pjayme 2018-06-25 11:31:46 +12:00 committed by Robbie Averill
parent a1a42009cc
commit 77f47af5cb
1 changed files with 23 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

44
javascript/CommentsInterface.js
View File

@ -6,31 +6,33 @@
/**
* Enable form validation
*/
$('.comments-holder-container form').validate({
$('.comments-holder-container form').each(function() {
$(this).validate({
// Ignore hidden elements in this form
ignore: ':hidden',
// Ignore hidden elements in this form
ignore: ':hidden',
// Use default 'required' for error labels
errorClass: "required",
// Use default 'required' for error labels
errorClass: "required",
// Use span instead of labels
errorElement: "span",
// Use span instead of labels
errorElement: "span",
// On error, scroll to the invalid element
invalidHandler : function(form, validator){
$('html, body').animate({
scrollTop: $(validator.errorList[0].element).offset().top - 30
}, 200);
},
// On error, scroll to the invalid element
invalidHandler : function(form, validator){
$('html, body').animate({
scrollTop: $(validator.errorList[0].element).offset().top - 30
}, 200);
},
// Ensure any new error message has the correct class and placement
errorPlacement: function(error, element) {
error
.addClass('message')
.insertAfter(element);
}
});
// Ensure any new error message has the correct class and placement
errorPlacement: function(error, element) {
error
.addClass('message')
.insertAfter(element);
}
});
});
/**
* Hide comment reply forms by default (unless visiting via permalink)
@ -56,7 +58,7 @@
// Prevent focus
e.preventDefault();
if(form.is(':visible')) {
allForms.slideUp();
} else {