From fe1708bb2c16693088bf33feefb290ac2dc3f511 Mon Sep 17 00:00:00 2001
From: Thomas Steen Rasmussen <tykling@bornhack.org>
Date: Sun, 2 Apr 2017 19:54:06 +0200
Subject: [PATCH] show team member count, only show join team button when team
 needs members

---
 src/teams/templates/team_detail.html | 4 ++--
 src/teams/templates/team_list.html   | 8 +++++++-
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/teams/templates/team_detail.html b/src/teams/templates/team_detail.html
index 0857b56d..0c11db4a 100644
--- a/src/teams/templates/team_detail.html
+++ b/src/teams/templates/team_detail.html
@@ -14,10 +14,10 @@ Team: {{ team.name }} | {{ block.super }}
 
 <hr />
 <h3>{{ team.area.name }} Area:</h3>
-<p>This team is under the <b>{{ team.area.name }} area</b>. The following users are responsible for the <b>{{ team.area.name }} area</b>:</p>
+<p>This team is under the <b>{{ team.area.name }} area</b>. The following people are responsible for the <b>{{ team.area.name }} area</b>:</p>
 <ul>
     {% for resp in team.area.responsible.all %}
-    <li>{{ resp.get_full_name|default:"Unnamed" }}</li>
+    <li><b>{{ resp.get_full_name|default:"Unnamed" }}</b></li>
     {% endfor %}
 </ul>
 {% endblock %}
diff --git a/src/teams/templates/team_list.html b/src/teams/templates/team_list.html
index 70c7e73b..bd12b9a6 100644
--- a/src/teams/templates/team_list.html
+++ b/src/teams/templates/team_list.html
@@ -15,6 +15,7 @@ Teams | {{ block.super }}
     <th>Description</th>
     <th>Area</th>
     <th>Responsible</th>
+    <th>Members</th>
     {% if request.user.is_authenticated %}
     <th>Membership</th>
     <th>Action</th>
@@ -26,7 +27,7 @@ Teams | {{ block.super }}
     <tr>
       <td>
         <a href="{% url 'team_detail' camp_slug=camp.slug slug=team.slug %}">
-          {{ team.name }}
+          {{ team.name }} Team
         </a>
       </td>
       <td>
@@ -40,6 +41,9 @@ Teams | {{ block.super }}
         {{ resp.get_full_name|default:"Unnamed" }}{% if not forloop.last %},{% endif %}<br>
         {% endfor %}
       </td>
+      <td class="text-center">
+        <span class="badge">{{ team.members.count }}</span>
+      </td>
       {% if request.user.is_authenticated %}
       <td>
         {% if request.user in team.members.all %}
@@ -52,8 +56,10 @@ Teams | {{ block.super }}
         {% if request.user in team.members.all %}
         <a href="{% url 'team_leave' camp_slug=camp.slug slug=team.slug %}" class="btn btn-danger"><i class="fa fa-minus"></i> Leave</a>
         {% else %}
+        {% if team.needs_members %}
         <a href="{% url 'team_join' camp_slug=camp.slug slug=team.slug %}" class="btn btn-success"><i class="fa fa-plus"></i> Join</button>
         {% endif %}
+        {% endif %}
       </td>
       {% endif %}
     </tr>