set styling for validation error lists

views/html/achievements/create.tpl

@@ -9,8 +9,8 @@
 </ul>
 
 <h1><?=_('Create Achievement')?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/achievements/edit.tpl

@@ -9,8 +9,8 @@
 </ul>
 
 <h1><?=_('Edit Achievement')?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/charactergroups/creategroup.tpl

@@ -10,8 +10,8 @@
 </ul>
 
 <h1><?=sprintf(_('New %s Character group'), $groupsgroup['name'])?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/charactergroups/creategroupsgroup.tpl

@@ -9,8 +9,8 @@
 </ul>
 
 <h1><?=_('New Character groups-group')?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/charactergroups/editgroup.tpl

@@ -10,8 +10,8 @@
 </ul>
 
 <h1><?=sprintf(_('Edit %s Character group'), $groupsgroup['name'])?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/charactergroups/editgroupsgroup.tpl

@@ -9,8 +9,8 @@
 </ul>
 
 <h1><?=_('Edit Character groups-group')?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/charactergroupsquests/create.tpl

@@ -10,8 +10,8 @@
 </ul>
 
 <h1><?=sprintf(_('New %s-Quest'), $groupsgroup['name'])?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/charactergroupsquests/edit.tpl

@@ -10,8 +10,8 @@
 </ul>
 
 <h1><?=sprintf(_('Edit %s-Quest'), $groupsgroup['name'])?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/charactergroupsquests/manage.tpl

@@ -24,8 +24,8 @@
 
 <section>
 	<h1><i class="fa fa-picture-o fa-fw"></i><?=_('Media')?></h1>
-	<?php if($validation !== true) : ?>
-	<ul>
+	<?php if($validation !== true && !empty($validation)) : ?>
+	<ul class="validation">
 		<?php foreach($validation as $field => &$settings) : ?>
 		<li>
 			<ul>

views/html/characters/edit.tpl

@@ -17,8 +17,8 @@
 
 <h1><?=_('Edit Character')?></h1>
 <form method="post" action="" class="logreg">
-	<?php if($validation !== true) : ?>
-	<ul>
+	<?php if($validation !== true && !empty($validation)) : ?>
+	<ul class="validation">
 		<?php foreach($validation as $field => &$settings) : ?>
 		<li>
 			<ul>

views/html/characters/register.tpl

@@ -9,8 +9,8 @@
 </ul>
 <h1><?=_('Create Character')?></h1>
 <form method="post" action="" class="logreg">
-	<?php if($validation !== true) : ?>
-	<ul>
+	<?php if($validation !== true && !empty($validation)) : ?>
+	<ul class="validation">
 		<?php foreach($validation as $field => &$settings) : ?>
 		<li>
 			<ul>

views/html/charactertypes/create.tpl

@@ -9,8 +9,8 @@
 </ul>
 
 <h1><?=_('Create new Charactertype')?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/charactertypes/edit.tpl

@@ -9,8 +9,8 @@
 </ul>
 
 <h1><?=_('Edit Charactertype')?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/library/create.tpl

@@ -9,8 +9,8 @@
 </ul>
 <h1><i class="fa fa-book fa-fw"></i><?=_('New Questtopic')?></h1>
 
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/library/edit.tpl

@@ -10,7 +10,7 @@
 
 <h1><i class="fa fa-book fa-fw"></i><?=_('Edit Questtopic')?></h1>
 <?php if($validations['edit'] !== true) : ?>
-<ul>
+<ul class="validation">
 	<?php foreach($validations['edit'] as $field => &$settings) : ?>
 	<li>
 		<ul>
@@ -50,7 +50,7 @@
 <form method="post">
 	<?php foreach($subtopicsTitles as $subtopicId => &$title) : ?>
 	<?php if($validations['edit-subtopics'] !== true && array_key_exists($subtopicId, $validations['edit-subtopics']) && $validations['edit-subtopics'][$subtopicId] !== true) : ?>
-	<ul>
+	<ul class="validation">
 		<?php foreach($validations['edit-subtopics'][$subtopicId] as $field => &$settings) : ?>
 		<li>
 			<ul>
@@ -88,7 +88,7 @@
 
 <h2><?=_('Create new Questsubtopic')?></h2>
 <?php if($validations['create-subtopic'] !== true) : ?>
-<ul>
+<ul class="validation">
 	<?php foreach($validations['create-subtopic'] as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/questgroups/create.tpl

@@ -9,8 +9,8 @@
 </ul>
 
 <h1><?=_('Create Questgroup')?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/questgroups/edit.tpl

@@ -6,8 +6,8 @@
 <?=$questgroupshierarchypath?>
 
 <h1><?=_('Edit Questgroup')?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/questgroupshierarchy/create.tpl

@@ -9,8 +9,8 @@
 </ul>
 
 <h1><?=_('Create Questgroupshierarchy')?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/questgroupshierarchy/edit.tpl

@@ -9,8 +9,8 @@
 </ul>
 
 <h1><?=_('Edit Questgroupshierarchy')?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/quests/create.tpl

@@ -6,8 +6,8 @@
 <?=$questgroupshierarchypath?>
 
 <h1><?=_('Create Quest')?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/quests/edit.tpl

@@ -6,8 +6,8 @@
 <?=$questgroupshierarchypath?>
 
 <h1><?=_('Edit Quest')?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>

views/html/quests/edittexts.tpl

@@ -11,7 +11,7 @@
 	<fieldset>
 		<legend><?=$type['type']?></legend>
 		<?php if(array_key_exists($type['url'], $validations)) : ?>
-		<ul>
+		<ul class="validation">
 			<?php foreach($validations[$type['url']] as $textId => &$validation) : ?>
 			<?php if($validation !== true) : ?>
 				<?php foreach($validation as $field => &$settings) : ?>

views/html/seminaries/copy.tpl

@@ -17,8 +17,8 @@
 	<?php endif ?>
 </p>
 <?php endif ?>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>
@@ -67,7 +67,7 @@
 <form method="post" class="logreg" enctype="multipart/form-data">
 	<fieldset>
 		<label for="title"><?=_('Title')?>:</label>
-		<input type="text" id="title" name="title" placeholder="<?=_('Title')?>" maxlength="<?=$validationSettings['title']['maxlength']?>" value="<?=$title?>" <?=($validation !== true && array_key_exists('title', $validation)) ? 'class="invalid"' : null?> /><br />
+		<input type="text" id="title" name="title" placeholder="<?=_('Title')?>" required="required" maxlength="<?=$validationSettings['title']['maxlength']?>" value="<?=$title?>" <?=($validation !== true && array_key_exists('title', $validation)) ? 'class="invalid"' : null?> /><br />
 		<label for="course"><?=_('Course')?>:</label>
 		<input type="text" id="course" name="course" placeholder="<?=_('Course')?>" maxlength="<?=$validationSettings['course']['maxlength']?>" value="<?=$course?>" <?=($validation !== true && array_key_exists('course', $validation)) ? 'class="invalid"' : null?> /><br />
 		<label for="description"><?=_('Description')?>:<br />

views/html/seminaries/create.tpl

@@ -6,8 +6,8 @@
 </ul>
 
 <h1><?=_('New seminary')?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>
@@ -66,7 +66,7 @@
 	</fieldset>
 	<fieldset>
 		<label for="title"><?=_('Title')?>:</label>
-		<input type="text" name="title" placeholder="<?=_('Title')?>" maxlength="<?=$validationSettings['title']['maxlength']?>" value="<?=$title?>" <?=(array_key_exists('title', $validation)) ? 'class="invalid"' : null?> /><br />
+		<input type="text" name="title" placeholder="<?=_('Title')?>" required="required" maxlength="<?=$validationSettings['title']['maxlength']?>" value="<?=$title?>" <?=(array_key_exists('title', $validation)) ? 'class="invalid"' : null?> /><br />
 		<label for="course"><?=_('Course')?>:</label>
 		<input type="text" name="course" placeholder="<?=_('Course')?>" maxlength="<?=$validationSettings['course']['maxlength']?>" value="<?=$course?>" <?=(array_key_exists('course', $validation)) ? 'class="invalid"' : null?> /><br />
 		<label for="description"><?=_('Description')?>:<br />

views/html/seminaries/edit.tpl

@@ -8,8 +8,8 @@
 </ul>
 
 <h1><?=_('Edit seminary')?></h1>
-<?php if($validation !== true) : ?>
-<ul>
+<?php if($validation !== true && !empty($validation)) : ?>
+<ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
 		<ul>
@@ -68,7 +68,7 @@
 	</fieldset>
 	<fieldset>
 		<label for="title"><?=_('Title')?>:</label>
-		<input type="text" name="title" placeholder="<?=_('Title')?>" maxlength="<?=$validationSettings['title']['maxlength']?>" value="<?=$title?>" <?=(array_key_exists('title', $validation)) ? 'class="invalid"' : null?> /><br />
+		<input type="text" name="title" placeholder="<?=_('Title')?>" required="required" maxlength="<?=$validationSettings['title']['maxlength']?>" value="<?=$title?>" <?=(array_key_exists('title', $validation)) ? 'class="invalid"' : null?> /><br />
 		<label for="course"><?=_('Course')?>:</label>
 		<input type="text" name="course" placeholder="<?=_('Course')?>" maxlength="<?=$validationSettings['course']['maxlength']?>" value="<?=$course?>" <?=(array_key_exists('course', $validation)) ? 'class="invalid"' : null?> /><br />
 		<label for="description"><?=_('Description')?>:<br />

views/html/users/register.tpl

@@ -6,11 +6,11 @@
 </ul>
 
 <h1><?=_('Registration')?></h1>
-<?php if($validation !== true) : ?>
+<?php if($validation !== true && !empty($validation)) : ?>
 <ul class="validation">
 	<?php foreach($validation as $field => &$settings) : ?>
 	<li>
-		<ul class="subvalidation">
+		<ul>
 			<?php foreach($settings as $setting => $value) : ?>
 			<li>
 				<?php switch($field) {

views/html/xplevels/manage.tpl

@@ -14,7 +14,7 @@
 		<?php foreach($xplevels as &$xplevel) : ?>
 		<li>
 			<?php if($validations['edit'] !== true && array_key_exists($xplevel['id'], $validations['edit']) && $validations['edit'][$xplevel['id']] !== true) : ?>
-			<ul>
+			<ul class="validation">
 				<?php foreach($validations['edit'][$xplevel['id']] as $field => &$settings) : ?>
 				<li>
 					<ul>
@@ -51,7 +51,7 @@
 
 <h2><?=_('Create new XP-level')?></h2>
 <?php if($validations['create'] !== true) : ?>
-<ul>
+<ul class="validation">
 	<?php foreach($validations['create'] as $field => &$settings) : ?>
 	<li>
 		<ul>

www/css/desktop.css

@@ -213,6 +213,7 @@ input[type="submit"][disabled]{text-shadow:1px 2px #d48c4e;background:#f9ac69;bo
 .avatar .admin li{display:block;margin:10px}
 
 .invalid{background:#ebd3d3;color:#850000}
+.validation{list-style-type:circle;color:#850000;padding:1em 1em 1em 2em;border-radius:1em;background:#ebd3d3}
 
 
 /** Character Profile **/