Clean up team index page.

Changes:
- the striped tables don't provide a lot of additional value (anymore?),
  and I think the gray bites with table header - perhaps do the same
  changes later on other pages
- The bold-ing of the submissions that have not been clicked is
  unnecessary - there is no real additional content that we want teams
  to acknowledge. We probably should use the unread column in the
  database for notifications instead
- Remove the from/to columns for the clarification side and also remove
  the word "problem" in the subject column. from/to makes sense for jury
  interface but not so much for teams.

Author: meisterT

webapp/templates/team/partials/clarification_list.html.twig

@@ -4,12 +4,10 @@
 {% endif %}
 
 <div class="table-wrapper">
-    <table class="data-table table table-striped table-hover table-sm" style="width:100%;">
+    <table class="data-table table table-hover table-sm" style="width:100%;">
         <thead class="thead-light">
         <tr>
             <th scope="col">time</th>
-            <th scope="col">from</th>
-            <th scope="col">to</th>
             {% if includeSubject %}
                 <th scope="col">subject</th>
             {% endif %}
@@ -26,33 +24,16 @@
                         {{ clarification.submittime | printtime(null, clarification.contest) }}
                     </a>
                 </td>
-                {%- if clarification.sender is null %}
-                    {%- set sender = 'Jury' %}
-                    {%- if clarification.recipient is null %}
-                        {%- set recipient = 'All' %}
-                    {%- else %}
-                        {%- set recipient = clarification.recipient.effectiveName %}
-                    {%- endif %}
-                {%- else %}
-                    {%- set recipient = 'Jury' %}
-                    {%- set sender = clarification.sender.effectiveName %}
+                {%- set prefix = '' %}
+                {%- if clarification.sender is null and clarification.recipient is not null %}
+                    {%- set prefix = 'Reply to ' ~ clarification.recipient.effectiveName ~ ': ' %}
                 {%- endif %}
 
-                <td>
-                    <a data-ajax-modal data-ajax-modal-after="markSeen" href="{{ link }}" title="{{ sender }}">
-                        {{ sender | u.truncate(teamname_max_length, '…') }}
-                    </a>
-                </td>
-                <td>
-                    <a data-ajax-modal data-ajax-modal-after="markSeen" href="{{ link }}" title="{{ recipient }}">
-                        {{ recipient | u.truncate(teamname_max_length, '…') }}
-                    </a>
-                </td>
                 {% if includeSubject %}
                     <td>
                         <a data-ajax-modal data-ajax-modal-after="markSeen" href="{{ link }}">
                             {%- if clarification.problem -%}
-                                problem {{ clarification.contestProblem | problemBadge -}}
+                                {{ clarification.contestProblem | problemBadge -}}
                             {%- elseif clarification.category -%}
                                 {{- categories[clarification.category]|default('general') -}}
                             {%- else -%}
@@ -64,7 +45,12 @@
 
                 <td>
                     <a data-ajax-modal data-ajax-modal-after="markSeen" href="{{ link }}">
-                        {{ clarification.summary | markdown_to_html | sanitize_html('app.clarification_sanitizer') }}
+                        {{ prefix }} <i>{{ clarification.summary
+                            | markdown_to_html
+                            | replace({'<p>': '', '</p>': ''})
+                            | sanitize_html('app.clarification_sanitizer')
+                            | raw
+                            }}</i>
                     </a>
                 </td>
             </tr>

webapp/templates/team/partials/submission_list.html.twig

@@ -4,7 +4,7 @@
 {% if submissions is empty %}
     <div class="alert alert-warning">No submissions</div>
 {% else %}
-    <table class="data-table table table-hover table-striped table-sm submissions-table">
+    <table class="data-table table table-hover table-sm submissions-table">
         <thead class="thead-light">
         <tr>
             <th scope="col">time</th>
@@ -30,9 +30,6 @@
             {% if not submission.valid %}
                 {% set classes = classes ~ ' ignore' %}
             {% endif %}
-            {% if submission.judgings.first is empty or not submission.judgings.first.seen %}
-                {% set classes = classes ~ ' unseen' %}
-            {% endif %}
             <tr class="{{ classes }}">
                 <td>
                     <a data-ajax-modal data-ajax-modal-after="markSeen" {% if link %}href="{{ link }}"{% endif %}>
@@ -81,7 +78,7 @@
         {% endif %}
         {% if allowDownload %}
                 <td>
-                    <a title="Download submission ZIP" aria-label="download submission zip" class="btn btn-light" href="{{ path('team_submission_download', {'submitId': submission.submitid}) }}">
+                    <a title="Download submission ZIP" aria-label="download submission zip" href="{{ path('team_submission_download', {'submitId': submission.submitid}) }}">
                         <i class="fa fa-file-archive"></i>
                     </a>
                 </td>