From 0914297428ac58a31171c954faa41ecd2a0ac64c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jordi=20Puiggen=C3=A9?= <jp@naralabs.com>
Date: Fri, 1 Sep 2023 14:05:22 +0200
Subject: [PATCH 1/4] Show range comment in results report

---
 .../analysisrequest/templates/results.pt      | 23 ++++++++++++++-----
 1 file changed, 17 insertions(+), 6 deletions(-)

diff --git a/src/senaite/impress/analysisrequest/templates/results.pt b/src/senaite/impress/analysisrequest/templates/results.pt
index 2482dde6..a5679cdf 100644
--- a/src/senaite/impress/analysisrequest/templates/results.pt
+++ b/src/senaite/impress/analysisrequest/templates/results.pt
@@ -47,7 +47,7 @@
                     <span i18n:translate="">Range</span>
                   </th>
                   <th class="outofrange">
-                    <span i18n:translate=""></span>
+                    <span/>
                   </th>
                 </tr>
               </thead>
@@ -93,14 +93,25 @@
                       <span tal:content="python:'(RT)' if model.is_retest(analysis) else ''"></span>
                       <span tal:content="python:model.get_formatted_specs(analysis)">50 - 60</span>
                     </td>
-                    <td class="text-center align-middle">
-                      <span tal:condition="python:model.is_out_of_range(analysis)"
-                            class="font-weight-light">
-                        <span class="outofrange text-danger"
+
+                    <!-- Out of range column -->
+                    <td class="text-center align-middle"
+                        tal:define="out_of_range  python: model.is_out_of_range(analysis);
+                                    results_range python: analysis.getResultsRange();
+                                    comment       python: results_range.get('rangecomment');">
+                      <tal:out_of_range condition="out_of_range">
+
+                        <!-- Out of range symbol -->
+                        <span class="font-weight-light outofrange text-danger"
                               style="font-family:Lucida Console, Courier, monospace;"
                               tal:content="outofrange_symbol">
                         </span>
-                      </span>
+
+                        <!-- Out-of-range comment -->
+                        <span tal:condition="comment" tal:content="comment"/>
+
+                      </tal:out_of_range>
+
                     </td>
                   </tr>
                 </tal:analyses>

From de1ff402910475267fd1379f6ff6e455ac192481 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jordi=20Puiggen=C3=A9?= <jp@naralabs.com>
Date: Fri, 1 Sep 2023 14:10:34 +0200
Subject: [PATCH 2/4] Better styling

---
 src/senaite/impress/analysisrequest/templates/results.pt | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/senaite/impress/analysisrequest/templates/results.pt b/src/senaite/impress/analysisrequest/templates/results.pt
index a5679cdf..20792878 100644
--- a/src/senaite/impress/analysisrequest/templates/results.pt
+++ b/src/senaite/impress/analysisrequest/templates/results.pt
@@ -108,7 +108,8 @@
                         </span>
 
                         <!-- Out-of-range comment -->
-                        <span tal:condition="comment" tal:content="comment"/>
+                        <span class="font-weight-light text-danger small"
+                              tal:condition="comment" tal:content="comment"/>
 
                       </tal:out_of_range>
 

From 4a8764d5303ade34f54e2878745ad3fe189cef87 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jordi=20Puiggen=C3=A9?= <jp@naralabs.com>
Date: Fri, 1 Sep 2023 14:11:21 +0200
Subject: [PATCH 3/4] Better text styling (2)

---
 src/senaite/impress/analysisrequest/templates/results.pt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/senaite/impress/analysisrequest/templates/results.pt b/src/senaite/impress/analysisrequest/templates/results.pt
index 20792878..844c961b 100644
--- a/src/senaite/impress/analysisrequest/templates/results.pt
+++ b/src/senaite/impress/analysisrequest/templates/results.pt
@@ -108,7 +108,7 @@
                         </span>
 
                         <!-- Out-of-range comment -->
-                        <span class="font-weight-light text-danger small"
+                        <span class="text-danger small"
                               tal:condition="comment" tal:content="comment"/>
 
                       </tal:out_of_range>

From b688158692086312f8a7931256ad3b7dd01650f8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jordi=20Puiggen=C3=A9?= <jp@naralabs.com>
Date: Fri, 1 Sep 2023 14:11:58 +0200
Subject: [PATCH 4/4] Changelog

---
 docs/Changelog.rst | 1 +
 1 file changed, 1 insertion(+)

diff --git a/docs/Changelog.rst b/docs/Changelog.rst
index c9bca6c1..d1491414 100644
--- a/docs/Changelog.rst
+++ b/docs/Changelog.rst
@@ -1,6 +1,7 @@
 2.5.0 (unreleased)
 ------------------
 
+- #142 Display custom comment for out of range results
 - #141 Display reportable interim fields as result variables in results report
 - #140 Refactor report sections into separate components