fix: Don't make a <tbody> for each row (ietf-tools#3808)

larseggert · web-flow · commit 4f1dcd6f4165 · 2022-04-12T12:04:57.000-03:00
Fixes ietf-tools#3805.
diff --git a/ietf/templates/stats/review_stats.html b/ietf/templates/stats/review_stats.html
@@ -174,11 +174,8 @@ <h2>Completion status and completion time</h2>
                 </tr>
             </thead>
             {% for row in data %}
-                    {% if forloop.revcounter != 1 %}<tbody>{% endif %}
-                    {% if forloop.last %}
-                        {% if forloop.revcounter != 1 %}</tbody>{% endif %}
-                        <tfoot>
-                    {% endif %}
+                    {% if forloop.first %}<tbody>{% endif %}
+                    {% if forloop.last %}</tbody><tfoot>{% endif %}
                     <tr>
                         <td>{{ row.obj }}</td>
                         <td>{{ row.open_in_time }}</td>
@@ -196,8 +193,8 @@ <h2>Completion status and completion time</h2>
                             {% endif %}
                         </td>
                     </tr>
-                {% endfor %}
-                </tfoot>
+                {% if forloop.last %}</tfoot>{% endif %}
+            {% endfor %}
             </table>
         {% elif stats_type == "results" %}
             <h2>
@@ -221,11 +218,8 @@ <h2>
                     </tr>
                 </thead>
                 {% for row in data %}
-                        {% if forloop.revcounter != 1 %}<tbody>{% endif %}
-                        {% if forloop.last %}
-                            {% if forloop.revcounter != 1 %}</tbody>{% endif %}
-                            <tfoot>
-                        {% endif %}
+                        {% if forloop.first %}<tbody>{% endif %}
+                        {% if forloop.last %}</tbody><tfoot>{% endif %}
                         <tr>
                             <td>
                                 {{ row.obj }}
@@ -236,8 +230,8 @@ <h2>
                                 </td>
                             {% endfor %}
                         </tr>
-                    {% endfor %}
-                </tfoot>
+                    {% if forloop.last %}</tfoot>{% endif %}
+                {% endfor %}
             </table>
         {% elif stats_type == "states" %}
             <h2>
@@ -261,11 +255,8 @@ <h2>
                     </tr>
                 </thead>
                 {% for row in data %}
-                    {% if forloop.revcounter != 1 %}<tbody>{% endif %}
-                        {% if forloop.last %}
-                            {% if forloop.revcounter != 1 %}</tbody>{% endif %}
-                            <tfoot>
-                        {% endif %}
+                    {% if forloop.first %}<tbody>{% endif %}
+                        {% if forloop.last %}</tbody><tfoot>{% endif %}
                         <tr>
                             <td>
                                 {{ row.obj }}
@@ -276,8 +267,8 @@ <h2>
                                 </td>
                             {% endfor %}
                         </tr>
-                    {% endfor %}
-                </tfoot>
+                    {% if forloop.last %}</tfoot>{% endif %}
+                {% endfor %}
             </table>
         {% elif stats_type == "time" and selected_teams %}
             <h2>
@@ -347,4 +338,4 @@ <h2>
             <script src="{% static 'ietf/js/review-stats.js' %}">
             </script>
         {% endif %}
-    {% endblock %}
+    {% endblock %}
