From db02edd7c8aebace8b32d358d9c7302f4e3732f1 Mon Sep 17 00:00:00 2001
From: Jan-Christoph Borchardt <hey@jancborchardt.net>
Date: Fri, 6 Mar 2015 15:27:21 +0100
Subject: [PATCH] adjust list of errors, more compact and readable

---
 settings/css/settings.css    | 8 +++++++-
 settings/js/admin.js         | 2 +-
 settings/templates/admin.php | 2 +-
 3 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/settings/css/settings.css b/settings/css/settings.css
index f7da029d7e..e2349e9dd6 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 9fe4226827..a3c941f08a 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 b6326108bf..132fd86a3f 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')));