From 479802968a818897dd5f6926e6d795d8dd519c40 Mon Sep 17 00:00:00 2001
From: FayCross <cross.fay@gmail.com>
Date: Fri, 4 Nov 2016 16:23:13 +0000
Subject: [PATCH] #547 Documentation page - minor change to required fields

---
 .../Nottingham/common_html5/css/mainStyles.css           | 6 ++++--
 .../Nottingham/models_html5/documentation.html           | 9 +++++++--
 2 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/modules/xerte/parent_templates/Nottingham/common_html5/css/mainStyles.css b/modules/xerte/parent_templates/Nottingham/common_html5/css/mainStyles.css
index d09ab98fb2..c02490baac 100644
--- a/modules/xerte/parent_templates/Nottingham/common_html5/css/mainStyles.css
+++ b/modules/xerte/parent_templates/Nottingham/common_html5/css/mainStyles.css
@@ -30,8 +30,6 @@ p {
     margin: 0;
 }
 
-
-
 #x_mainHolder {
 	width:		800px;
 	height:		600px;
@@ -381,6 +379,10 @@ audio {
 	color:			red;
 }
 
+.alertBorder {
+	border: 1px solid red;
+}
+
 fieldset.noStyle {
 	border: 0;
 	padding: 0;
diff --git a/modules/xerte/parent_templates/Nottingham/models_html5/documentation.html b/modules/xerte/parent_templates/Nottingham/models_html5/documentation.html
index 634f726cd2..c2b378b400 100644
--- a/modules/xerte/parent_templates/Nottingham/models_html5/documentation.html
+++ b/modules/xerte/parent_templates/Nottingham/models_html5/documentation.html
@@ -414,7 +414,7 @@
 						
 						$element = $('<div class="question optionGroup"/>').append($input).append(label);
 						break;
-					case 'media': // ** should work for audio / video too
+					case 'media':
 						if (xml.getAttribute('url').substr(0,10) == "USER-FILES") {
 							$element = $('<img class="itemImg">')
 								.attr('src', xml.getAttribute('url'));
@@ -427,6 +427,9 @@
 					case 'description':
 						break;
 					case 'line':
+						if (exclude != 'doc') {
+							docData.pages[page].sections[section].items[item].itemText = '<hr/>';
+						}
 						$element = $('<hr />');
 						break;
 					case 'selectlist': // radio,checkbox
@@ -572,6 +575,7 @@
 		};
 
 		this.checkRequired = function (page) {
+			$(".alertBorder").removeClass('alertBorder');
 			var ok = true;
 
 			for(var i=0; i<required.length; i++) {
@@ -580,6 +584,7 @@
 						case 'checkbox':
 							if (! $('#'+required[i].key).prop('checked')) {
 								ok = false;
+								$('#'+required[i].key).closest('.item').addClass('alertBorder');
 							}
 							break;
 						case 'textarea':
@@ -587,10 +592,10 @@
 						default:
 							if ($('#'+required[i].key).val() == '') {
 								ok = false;
+								$('#'+required[i].key).closest('.item').addClass('alertBorder');
 							}
 							break;
 					}
-					if (! ok) break;
 				}
 			}