From 838049cedefc5209ff9666624b2b24eccb120fb8 Mon Sep 17 00:00:00 2001
From: Frederick Muriuki Muriithi
Date: Wed, 16 Nov 2022 03:09:22 +0300
Subject: ui: Only display the "alert-success" green on completion

* To avoid confusion, only display the "alert-success" green on
  completion of the parsing process. While parsing, if there are no
  errors, then display the "No errors found so far" message without
  the green colour.
---
 README.org                           | 2 +-
 qc_app/templates/errors_display.html | 4 ++--
 qc_app/templates/job_progress.html   | 2 +-
 qc_app/templates/parse_results.html  | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/README.org b/README.org
index 5e5e0a9..a0202dc 100644
--- a/README.org
+++ b/README.org
@@ -31,7 +31,7 @@ For reproducibility, this project is developed using guix.
 
 To launch a guix shell for development, do
 #+BEGIN_SRC shell
-  guix shell --container --network --development --file=guix.scm --share=/some/host/directory=/the/upload/directory
+  guix shell --container --network --share=/some/host/directory=/the/upload/directory --development --file=guix.scm
 #+END_SRC
 which environment that is isolated from the rest of your system.
 
diff --git a/qc_app/templates/errors_display.html b/qc_app/templates/errors_display.html
index 39144a9..f1e27a6 100644
--- a/qc_app/templates/errors_display.html
+++ b/qc_app/templates/errors_display.html
@@ -1,7 +1,7 @@
-{%macro errors_display(errors, no_error_msg, error_message)%}
+{%macro errors_display(errors, no_error_msg, error_message, complete)%}
 
 {%if errors | length == 0 %}
-<span class="alert-success">{{no_error_msg}}</span>
+<span {%if complete%}class="alert-success"{%endif%}>{{no_error_msg}}</span>
 {%else %}
 <p class="alert-error">{{error_message}}</p>
 
diff --git a/qc_app/templates/job_progress.html b/qc_app/templates/job_progress.html
index acced9b..1b08814 100644
--- a/qc_app/templates/job_progress.html
+++ b/qc_app/templates/job_progress.html
@@ -25,7 +25,7 @@
 </div>
 
 <div>
-  {{errors_display(errors, "No errors found so far", "We have found the following errors so far")}}
+  {{errors_display(errors, "No errors found so far", "We have found the following errors so far", False)}}
 </div>
 
 {%endblock%}
diff --git a/qc_app/templates/parse_results.html b/qc_app/templates/parse_results.html
index 80d6a2d..ec19812 100644
--- a/qc_app/templates/parse_results.html
+++ b/qc_app/templates/parse_results.html
@@ -10,7 +10,7 @@
 <span class="alert-warning">Job aborted by the user</span>
 {%endif%}
 
-{{errors_display(errors, "No errors found in the file", "We found the following errors")}}
+{{errors_display(errors, "No errors found in the file", "We found the following errors", True)}}
 
 {%if errors | length == 0 %}
 <form method="post" action="{{url_for('dbinsert.select_platform')}}">
-- 
cgit v1.2.3