refine tab header

Signed-off-by: tdruez <tdruez@aboutcode.org>

Author: tdruez

product_portfolio/templates/product_portfolio/tabs/tab_triage.html

@@ -9,8 +9,7 @@
   {% endif %}
 </div>
 
-{% if page_obj.object_list %}
-  <table class="table table-hover">
+<table class="table table-hover">
     <thead class="table-light">
       <tr>
         <th style="width: 38%">{% trans "Package" %}</th>
@@ -93,15 +92,6 @@
       {% endfor %}
     </tbody>
   </table>
-  {% if page_obj.paginator.num_pages > 1 %}
-    {% include 'pagination/object_list_pagination.html' with hx_target=tab_id_html %}
-  {% endif %}
-{% else %}
-  <div class="text-center py-5">
-    <i class="fa-solid fa-circle-check text-success fs-2"></i>
-    <div class="mt-2 fw-semibold">{% trans "No pending triage actions" %}</div>
-    <div class="text-body-secondary small mt-1">
-      {% trans "All packages are within acceptable risk parameters." %}
-    </div>
-  </div>
+{% if page_obj.paginator.num_pages > 1 %}
+  {% include 'pagination/object_list_pagination.html' with hx_target=tab_id_html %}
 {% endif %}

product_portfolio/views.py

@@ -676,11 +676,17 @@ def tab_triage(self):
             TriageRecord.objects.filter(product_package__product=product).primary_actions().count()
         )
 
-        badge_class = "bg-primary-subtle text-primary-emphasis"
-        if triage_count > 0:
-            badge_class = "bg-warning-subtle text-warning-emphasis"
+        if triage_count == 0:
+            label = 'Triage <span class="badge bg-secondary">0</span>'
+            return {
+                "label": mark_safe(label),
+                "fields": [],
+                "disabled": True,
+                "tooltip": "No pending triage actions for this product",
+            }
 
-        label = f'Triage <span class="badge {badge_class}">{triage_count}</span>'
+        badge = f'<span class="badge bg-primary-subtle text-primary-emphasis">{triage_count}</span>'
+        label = f"Triage {badge}"
 
         tab_view_url = product.get_url("tab_triage")
         if full_query_string := self.request.META["QUERY_STRING"]: