From 625ba60f134a4ba588affb8056cc11229c6bf54d Mon Sep 17 00:00:00 2001
From: oliver <coderkun@users.noreply.github.com>
Date: Sun, 19 Apr 2015 12:15:21 +0200
Subject: [PATCH] set styling for validation error lists

---
 views/html/achievements/create.tpl               | 4 ++--
 views/html/achievements/edit.tpl                 | 4 ++--
 views/html/charactergroups/creategroup.tpl       | 4 ++--
 views/html/charactergroups/creategroupsgroup.tpl | 4 ++--
 views/html/charactergroups/editgroup.tpl         | 4 ++--
 views/html/charactergroups/editgroupsgroup.tpl   | 4 ++--
 views/html/charactergroupsquests/create.tpl      | 4 ++--
 views/html/charactergroupsquests/edit.tpl        | 4 ++--
 views/html/charactergroupsquests/manage.tpl      | 4 ++--
 views/html/characters/edit.tpl                   | 4 ++--
 views/html/characters/register.tpl               | 4 ++--
 views/html/charactertypes/create.tpl             | 4 ++--
 views/html/charactertypes/edit.tpl               | 4 ++--
 views/html/library/create.tpl                    | 4 ++--
 views/html/library/edit.tpl                      | 6 +++---
 views/html/questgroups/create.tpl                | 4 ++--
 views/html/questgroups/edit.tpl                  | 4 ++--
 views/html/questgroupshierarchy/create.tpl       | 4 ++--
 views/html/questgroupshierarchy/edit.tpl         | 4 ++--
 views/html/quests/create.tpl                     | 4 ++--
 views/html/quests/edit.tpl                       | 4 ++--
 views/html/quests/edittexts.tpl                  | 2 +-
 views/html/seminaries/copy.tpl                   | 6 +++---
 views/html/seminaries/create.tpl                 | 6 +++---
 views/html/seminaries/edit.tpl                   | 6 +++---
 views/html/users/register.tpl                    | 4 ++--
 views/html/xplevels/manage.tpl                   | 4 ++--
 www/css/desktop.css                              | 1 +
 28 files changed, 58 insertions(+), 57 deletions(-)

diff --git a/views/html/achievements/create.tpl b/views/html/achievements/create.tpl
index e88b6612..fabd866a 100644
--- a/views/html/achievements/create.tpl
+++ b/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>
diff --git a/views/html/achievements/edit.tpl b/views/html/achievements/edit.tpl
index 93f939c8..763e7806 100644
--- a/views/html/achievements/edit.tpl
+++ b/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>
diff --git a/views/html/charactergroups/creategroup.tpl b/views/html/charactergroups/creategroup.tpl
index 35b37d74..16bb45cc 100644
--- a/views/html/charactergroups/creategroup.tpl
+++ b/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>
diff --git a/views/html/charactergroups/creategroupsgroup.tpl b/views/html/charactergroups/creategroupsgroup.tpl
index 1d0e84aa..b0fdc682 100644
--- a/views/html/charactergroups/creategroupsgroup.tpl
+++ b/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>
diff --git a/views/html/charactergroups/editgroup.tpl b/views/html/charactergroups/editgroup.tpl
index d8c7de70..14890e9f 100644
--- a/views/html/charactergroups/editgroup.tpl
+++ b/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>
diff --git a/views/html/charactergroups/editgroupsgroup.tpl b/views/html/charactergroups/editgroupsgroup.tpl
index 47e2ec91..5e800246 100644
--- a/views/html/charactergroups/editgroupsgroup.tpl
+++ b/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>
diff --git a/views/html/charactergroupsquests/create.tpl b/views/html/charactergroupsquests/create.tpl
index 7fbbf5f0..0c775a67 100644
--- a/views/html/charactergroupsquests/create.tpl
+++ b/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>
diff --git a/views/html/charactergroupsquests/edit.tpl b/views/html/charactergroupsquests/edit.tpl
index 8984a14c..eeed199d 100644
--- a/views/html/charactergroupsquests/edit.tpl
+++ b/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>
diff --git a/views/html/charactergroupsquests/manage.tpl b/views/html/charactergroupsquests/manage.tpl
index 126f8878..3d7bc8e6 100644
--- a/views/html/charactergroupsquests/manage.tpl
+++ b/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>
diff --git a/views/html/characters/edit.tpl b/views/html/characters/edit.tpl
index e5815d77..c882ebbf 100644
--- a/views/html/characters/edit.tpl
+++ b/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>
diff --git a/views/html/characters/register.tpl b/views/html/characters/register.tpl
index d0ecbf09..07bf2eca 100644
--- a/views/html/characters/register.tpl
+++ b/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>
diff --git a/views/html/charactertypes/create.tpl b/views/html/charactertypes/create.tpl
index 46a642af..1ad6b3c7 100644
--- a/views/html/charactertypes/create.tpl
+++ b/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>
diff --git a/views/html/charactertypes/edit.tpl b/views/html/charactertypes/edit.tpl
index 6bddfa40..26b4b31a 100644
--- a/views/html/charactertypes/edit.tpl
+++ b/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>
diff --git a/views/html/library/create.tpl b/views/html/library/create.tpl
index b1282434..8b62da53 100644
--- a/views/html/library/create.tpl
+++ b/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>
diff --git a/views/html/library/edit.tpl b/views/html/library/edit.tpl
index 2083eb76..a82580b8 100644
--- a/views/html/library/edit.tpl
+++ b/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>
diff --git a/views/html/questgroups/create.tpl b/views/html/questgroups/create.tpl
index e7e0c612..ad680204 100644
--- a/views/html/questgroups/create.tpl
+++ b/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>
diff --git a/views/html/questgroups/edit.tpl b/views/html/questgroups/edit.tpl
index 97c0f53e..974308f3 100644
--- a/views/html/questgroups/edit.tpl
+++ b/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>
diff --git a/views/html/questgroupshierarchy/create.tpl b/views/html/questgroupshierarchy/create.tpl
index 0d929b80..83734d97 100644
--- a/views/html/questgroupshierarchy/create.tpl
+++ b/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>
diff --git a/views/html/questgroupshierarchy/edit.tpl b/views/html/questgroupshierarchy/edit.tpl
index d82681df..4d353e12 100644
--- a/views/html/questgroupshierarchy/edit.tpl
+++ b/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>
diff --git a/views/html/quests/create.tpl b/views/html/quests/create.tpl
index 9e72d6a3..d9c58e49 100644
--- a/views/html/quests/create.tpl
+++ b/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>
diff --git a/views/html/quests/edit.tpl b/views/html/quests/edit.tpl
index adccac9a..7211c091 100644
--- a/views/html/quests/edit.tpl
+++ b/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>
diff --git a/views/html/quests/edittexts.tpl b/views/html/quests/edittexts.tpl
index e5ec350e..0bfd8397 100644
--- a/views/html/quests/edittexts.tpl
+++ b/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) : ?>
diff --git a/views/html/seminaries/copy.tpl b/views/html/seminaries/copy.tpl
index 39187f50..f1da13f9 100644
--- a/views/html/seminaries/copy.tpl
+++ b/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 />
diff --git a/views/html/seminaries/create.tpl b/views/html/seminaries/create.tpl
index b7b645f3..a2feddc9 100644
--- a/views/html/seminaries/create.tpl
+++ b/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 />
diff --git a/views/html/seminaries/edit.tpl b/views/html/seminaries/edit.tpl
index 4f0db564..90881973 100644
--- a/views/html/seminaries/edit.tpl
+++ b/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 />
diff --git a/views/html/users/register.tpl b/views/html/users/register.tpl
index f4d16887..680e0e1f 100644
--- a/views/html/users/register.tpl
+++ b/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) {
diff --git a/views/html/xplevels/manage.tpl b/views/html/xplevels/manage.tpl
index 4f37097b..867b3e4e 100644
--- a/views/html/xplevels/manage.tpl
+++ b/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>
diff --git a/www/css/desktop.css b/www/css/desktop.css
index 9125f7c9..56adcbc6 100644
--- a/www/css/desktop.css
+++ b/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 **/