Merge pull request #14736 from owncloud/errors-list

adjust list of errors, more compact and readable

3 files changed, 9 insertions, 3 deletions

diff --git a/settings/css/settings.css b/settings/css/settings.css
index f7da029d7e4..e2349e9dd68 100644
--- a/settings/css/settings.css
+++ b/settings/css/settings.css
@@ -339,6 +339,12 @@ doesnotexist:-o-prefocus, .strengthify-wrapper {
 	background-position: left center;
 }
 
-#postsetupchecks .hint, #postsetupchecks .setupwarning {
+#postsetupchecks .hint {
 	margin-top: 15px;
 }
+
+#postsetupchecks .setupwarning {
+	list-style: initial;
+	font-weight: normal;
+	margin-top: 5px;
+}
diff --git a/settings/js/admin.js b/settings/js/admin.js
index 9fe4226827c..a3c941f08a4 100644
--- a/settings/js/admin.js
+++ b/settings/js/admin.js
@@ -144,7 +144,7 @@ $(document).ready(function(){
 		} else {
 			$errorsEl = $el.find('.errors');
 			for (var i = 0; i < errors.length; i++ ) {
-				$errorsEl.append('<div class="setupwarning">' + errors[i] + '</div>');
+				$errorsEl.append('<li class="setupwarning">' + errors[i] + '</li>');
 			}
 			$errorsEl.removeClass('hidden');
 			$el.find('.hint').removeClass('hidden');
diff --git a/settings/templates/admin.php b/settings/templates/admin.php
index b6326108bf6..132fd86a3f3 100644
--- a/settings/templates/admin.php
+++ b/settings/templates/admin.php
@@ -225,7 +225,7 @@ if ($_['cronErrors']) {
 	<h2><?php p($l->t('Configuration Checks'));?></h2>
 	<div class="loading"></div>
 	<div class="success hidden"><?php p($l->t('No problems found'));?></div>
-	<div class="errors hidden"></div>
+	<ul class="errors hidden"></ul>
 	<div class="hint hidden">
 		<span class="setupwarning"><?php
 			print_unescaped($l->t('Please double check the <a href=\'%s\'>installation guides</a>.', \OC_Helper::linkToDocs('admin-install')));