From 797ae744331d3e2b4b0c8ef9d9b752c36df37349 Mon Sep 17 00:00:00 2001
From: Krishnan Shankar <krishnans2006@gmail.com>
Date: Wed, 8 Jan 2025 23:24:42 -0500
Subject: [PATCH] Improve student list UI for assignments

---
 tin/templates/assignments/show.html | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/tin/templates/assignments/show.html b/tin/templates/assignments/show.html
index 49b74127..4d8fee32 100644
--- a/tin/templates/assignments/show.html
+++ b/tin/templates/assignments/show.html
@@ -129,7 +129,7 @@ <h2 style="border-top:1px solid lightgray;padding-top:15px;">Filter Submissions<
       {% if assignment.is_quiz %}
         <table id="submission-list" class="has-border">
           <tr>
-            <th style="min-width:20px"></th>
+            <th style="min-width:20px; text-align: center;">#</th>
             <th style="min-width:125px">Student</th>
             {% if not active_period.name %}
               <th style="min-width:65px;">Period</th>
@@ -142,7 +142,7 @@ <h2 style="border-top:1px solid lightgray;padding-top:15px;">Filter Submissions<
           </tr>
           {% for student, period, latest_submission, graded_submission, ended, quiz_issues in students_and_submissions %}
             <tr>
-              <td style="text-align: center">{{ forloop.counter }}</td>
+              <td style="text-align: center;">{{ forloop.counter }}</td>
               <td><a href="{% url 'assignments:student_submission' assignment.id student.id %}">{{ student.full_name }}
                 ({{ student.username }})</a></td>
               {% if not active_period.name %}
@@ -164,14 +164,14 @@ <h2 style="border-top:1px solid lightgray;padding-top:15px;">Filter Submissions<
 
           {% if not students_and_submissions %}
             <tr>
-              <td colspan="{% if not active_period.name %}7{% else %}6{% endif %}" class="italic center">No students in class</td>
+              <td colspan="{% if not active_period.name %}8{% else %}7{% endif %}" class="italic center">No matching students in class</td>
             </tr>
           {% endif %}
         </table>
       {% else %}
         <table id="submission-list" class="has-border">
           <tr>
-            <th style="min-width:20px"></th>
+            <th style="min-width:20px; text-align:center;">#</th>
             <th style="min-width:125px">Student</th>
             {% if not active_period.name %}
               <th style="min-width:65px;">Period</th>
@@ -185,7 +185,7 @@ <h2 style="border-top:1px solid lightgray;padding-top:15px;">Filter Submissions<
           </tr>
           {% for student, period, latest_submission, graded_submission, new_login, new_24 in students_and_submissions %}
             <tr>
-              <td style="text-align:center">{{ forloop.counter }}</td>
+              <td style="text-align:center;">{{ forloop.counter }}</td>
               <td><a href="{% url 'assignments:student_submission' assignment.id student.id %}">{{ student.full_name }}
                 ({{ student.username }})</a></td>
               {% if not active_period.name %}
@@ -213,7 +213,7 @@ <h2 style="border-top:1px solid lightgray;padding-top:15px;">Filter Submissions<
 
           {% if not students_and_submissions %}
             <tr>
-              <td colspan="{% if not active_period.name %}8{% else %}7{% endif %}" class="italic center">No students in class</td>
+              <td colspan="{% if not active_period.name %}9{% else %}8{% endif %}" class="italic center">No matching students in class</td>
             </tr>
           {% endif %}
         </table>