From fd6df8c4b07e4b560a65d3f391dd1a442c1dbf44 Mon Sep 17 00:00:00 2001
From: Santiago Lamora <slamora@users.noreply.github.com>
Date: Wed, 4 Dec 2019 12:37:35 +0100
Subject: [PATCH] Code styles of mailing list view.

---
 musician/models.py                            |  3 ++-
 musician/templates/musician/mailinglists.html | 17 ++++++++++++-----
 musician/views.py                             |  1 +
 3 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/musician/models.py b/musician/models.py
index 7544360..5b61e16 100644
--- a/musician/models.py
+++ b/musician/models.py
@@ -146,7 +146,8 @@ class MailService(OrchestraModel):
 
 class MailinglistService(OrchestraModel):
     api_name = 'mailinglist'
-    verbose_name = 'Mailing list'
+    verbose_name = _('Mailing list')
+    description = _('Litle description of what to be expected in this section to aid the user. Even a link to more help if there is one available.')
     fields = ('name', 'status', 'address_name', 'admin_email', 'configure')
     param_defaults = {
         'name': None,
diff --git a/musician/templates/musician/mailinglists.html b/musician/templates/musician/mailinglists.html
index 031c33e..e24ea7a 100644
--- a/musician/templates/musician/mailinglists.html
+++ b/musician/templates/musician/mailinglists.html
@@ -3,10 +3,17 @@
 
 {% block content %}
 
-<h1>Mailing lists</h1>
-<p>Little description of what to be expected...</p>
+<h1 class="service-name">{{ service.verbose_name }}</h1>
+<p class="service-description">{{ service.description }}</p>
 
-<table class="table table-hover">
+<table class="table service-list">
+    <colgroup>
+      <col span="1" style="width: 13%;">
+      <col span="1" style="width: 12%;">
+      <col span="1" style="width: 50%;">
+      <col span="1" style="width: 15%;">
+      <col span="1" style="width: 10%;">
+  </colgroup>
   <thead class="thead-dark">
     <tr>
       <th scope="col">Name</th>
@@ -20,8 +27,8 @@
     {% for resource in object_list %}
     <tr>
       <th scope="row">{{ resource.name }}</th>
-      <td>{{ resource.status }}</td>
-      <td>{{ resource.address_name}}@{{ resource.address_domain.name }}</td>
+      <td class="text-primary font-weight-bold">{{ resource.status|capfirst }}</td>
+      <td>{{ resource.address_name}}</td>
       <td>{{ resource.admin_email }}</td>
       <td><a href="#TODO-{{ resource.manager_url }}" target="_blank" rel="noopener noreferrer">Mailtrain</a></td>
     </tr>
diff --git a/musician/views.py b/musician/views.py
index 632f8ea..4f981e2 100644
--- a/musician/views.py
+++ b/musician/views.py
@@ -109,6 +109,7 @@ class MailView(ServiceListView):
 
 class MailingListsView(ServiceListView):
     service_class = MailinglistService
+    template_name = "musician/mailinglists.html"
 
 
 class DatabasesView(ServiceListView):