litvinovg/vitro
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

NIHVIVO-2232: Worked on error display for form validation on menu items

Browse source
This commit is contained in:
manolobevia 2011-07-15 21:36:27 +00:00
parent 526437e093
commit e3e95fb11b
3 changed files with 8 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
webapp/web/css/menupage/menuManagement.css
View file

@ -51,4 +51,7 @@ ul#selectedClasses {
} }
ul#selectedClasses li { ul#selectedClasses li {
margin-left: .4em; margin-left: .4em;
}
#error-alert{
height: auto !important;
} }

5
webapp/web/js/menupage/menumanagement_edit.js
View file

@ -49,6 +49,7 @@ var menuManagementEdit = {
} else{ } else{
$('#error-alert').removeClass('hidden'); $('#error-alert').removeClass('hidden');
$('#error-alert p').html(validationError); $('#error-alert p').html(validationError);
$.scrollTo({ top:0, left:0}, 500)
return false; return false;
} }
}); });
@ -111,7 +112,7 @@ var menuManagementEdit = {
if ($("input:radio[name=selectedTemplate]:checked").val() == "custom") { if ($("input:radio[name=selectedTemplate]:checked").val() == "custom") {
if($("input[name=customTemplate]").val() == "") { if($("input[name=customTemplate]").val() == "") {
validationError += " * Enter custom template <br />"; validationError += "You must supply a Template<br />";
} }
} }
@ -123,7 +124,7 @@ var menuManagementEdit = {
var noClassesSelected = $("input[name='classInClassGroup']:checked").length; var noClassesSelected = $("input[name='classInClassGroup']:checked").length;
if(noClassesSelected == 0) { if(noClassesSelected == 0) {
//at least one class should be selected //at least one class should be selected
validationError += "You must have at least one class selected <br />"; validationError += "You must supply some content for displaying <br />";
} }
} }

3
webapp/web/templates/freemarker/edit/forms/menuManagement.ftl
View file

@ -65,7 +65,7 @@
<section id="classesInSelectedGroup" name="classesInSelectedGroup" ${existingClassGroupStyle}> <section id="classesInSelectedGroup" name="classesInSelectedGroup" ${existingClassGroupStyle}>
<#-- Select classes in a class group --> <#-- Select classes in a class group -->
<p id="selectClassesMessage" name="selectClassesMessage">Select content to display</p> <p id="selectClassesMessage" name="selectClassesMessage">Select content to display<span class="requiredHint"> *</span></p>
<section id="internal-class" role="region"> <section id="internal-class" role="region">
<#if internalClass?has_content> <#if internalClass?has_content>
@ -115,4 +115,5 @@
${stylesheets.add('<link rel="stylesheet" href="${urls.base}/css/menupage/menuManagement.css" />')} ${stylesheets.add('<link rel="stylesheet" href="${urls.base}/css/menupage/menuManagement.css" />')}
${stylesheets.add('<link rel="stylesheet" href="${urls.base}/edit/forms/css/customForm.css" />')} ${stylesheets.add('<link rel="stylesheet" href="${urls.base}/edit/forms/css/customForm.css" />')}
${scripts.add('<script type="text/javascript" src="${urls.base}/js/jquery_plugins/jquery.scrollTo-min.js"></script>')}
${scripts.add('<script type="text/javascript" src="${urls.base}/js/menupage/menumanagement_edit.js"></script>')} ${scripts.add('<script type="text/javascript" src="${urls.base}/js/menupage/menumanagement_edit.js"></script>')}