From fdad7e1f8ee6ab86b24d79f1ec0911b432b1d416 Mon Sep 17 00:00:00 2001
From: Frederick Muriuki Muriithi
Date: Tue, 3 Dec 2024 15:05:44 -0600
Subject: Improve UI: Move if check out of table

Create a new section for the errors display, and only display the
table if there is at least one error.
---
 uploader/templates/phenotypes/job-status.html | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

(limited to 'uploader')

diff --git a/uploader/templates/phenotypes/job-status.html b/uploader/templates/phenotypes/job-status.html
index cfbedd5..8ba3edf 100644
--- a/uploader/templates/phenotypes/job-status.html
+++ b/uploader/templates/phenotypes/job-status.html
@@ -40,9 +40,15 @@
   {%endif%}
 </div>
 
-<div class="row" style="min-height: 3em; max-height: 30em; overflow: auto;">
+<h4 class="subheading">Errors</h4>
+<div class="row" style="max-height: 20em; overflow: auto;">
+  {%if errors | length == 0 %}
+  <p class="text-info">
+    <span class="glyphicon glyphicon-info-sign"></span>
+    No errors found so far
+  </p>
+  {%else%}
   <table class="table">
-    <caption>Errors</caption>
     <thead>
       <tr>
         <th>File</th>
@@ -58,17 +64,14 @@
         <td>{{error.rowtitle}}</td>
         <td>{{error.coltitle}}</td>
         <td>{{error.message}}</td>
-      </tr>
-      {%else%}
-      <tr>
-        <td colspan="4" class="text-info">
-          No errors found so far.
         </td>
       </tr>
       {%endfor%}
     </tbody>
   </table>
+  {%endif%}
 </div>
+
 <div class="row">
   {{cli_output(job, "stdout")}}
 </div>
@@ -76,6 +79,7 @@
 <div class="row">
   {{cli_output(job, "stderr")}}
 </div>
+
 {%else%}
 <div class="row">
   <h3 class="text-danger">No Such Job</h3>
-- 
cgit v1.2.3