From 976d88efb19f336887a5e9243c9096fc1003cc49 Mon Sep 17 00:00:00 2001
From: Bill Scales <bill_scales@uk.ibm.com>
Date: Thu, 14 Aug 2025 13:31:42 +0100
Subject: [PATCH] Improve rendering of multi-line job failure reasons

Modify css code-text to preserve white space so
a multi-line failure reason retains its formatting
and update where it is used to display the job failure
reason to avoid adding whitespace

Signed-off-by: Bill Scales <bill_scales@uk.ibm.com>
---
 public/css/main.css                   | 1 +
 pulpito/templates/job.html            | 4 +---
 pulpito/templates/run_jobs_table.html | 4 +---
 3 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/public/css/main.css b/public/css/main.css
index f5ae181..be6f59d 100644
--- a/public/css/main.css
+++ b/public/css/main.css
@@ -50,6 +50,7 @@ table.link-bare td a:hover {
     font-family: 'UbuntuMono';
     background-color: #eee;
     padding-left: 25px;
+    white-space: pre-line;
 }
 
 /* Progress meter */
diff --git a/pulpito/templates/job.html b/pulpito/templates/job.html
index 8f13a43..78b6c44 100644
--- a/pulpito/templates/job.html
+++ b/pulpito/templates/job.html
@@ -119,9 +119,7 @@ <h1 id="tables"><a href="/{{ job.name }}/">{{ job.name }}</a><span class='spaced
               {% endif %}
               {% if job.failure_reason %}
                 <h4> Failure Reason:</h4>
-                <p class="code-text">
-                    {{ job.failure_reason|e }}
-                </p>
+                <p class="code-text">{{ job.failure_reason|e }}</p>
               {% endif %}
             </div>
             <div class="panel-group" id="detail-panel">
diff --git a/pulpito/templates/run_jobs_table.html b/pulpito/templates/run_jobs_table.html
index 2ff6a50..43eba67 100644
--- a/pulpito/templates/run_jobs_table.html
+++ b/pulpito/templates/run_jobs_table.html
@@ -93,9 +93,7 @@
           <tr class="tablesorter-childRow job_fail_extra">
             <td colspan="14" class="hiddenRow"><div class="collapse" id="{{ job.job_id }}">
               Failure Reason:
-              <p class="code-text">
-                {{ job.failure_reason|e }}
-              </p>
+              <p class="code-text">{{ job.failure_reason|e }}</p>
             </div></td>
           </tr>
         {% endif %}