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/src/Entity/Clarification.php

@@ -352,6 +352,6 @@ public function getSummary(): string
                 $newBody .= $line . ' ';
             }
         }
-        return Utils::cutString(html_entity_decode((empty($newBody) ? $this->getBody() : $newBody)), 80);
+        return Utils::cutString(html_entity_decode((empty($newBody) ? $this->getBody() : $newBody)), 120);
     }
 }

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,12 @@
                         {{ 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 %}
-                {%- 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 +41,15 @@
 
                 <td>
                     <a data-ajax-modal data-ajax-modal-after="markSeen" href="{{ link }}">
-                        {{ clarification.summary | markdown_to_html | sanitize_html('app.clarification_sanitizer') }}
+                        {%- if clarification.sender is null and clarification.recipient is not null %}
+                            <i class="fa-solid fa-reply" title="Reply to you"></i>&nbsp;
+                        {%- endif %}
+                        <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>

webapp/tests/Unit/Controller/Team/ClarificationControllerTest.php

@@ -23,7 +23,7 @@ public function testClarificationRequest(): void
         $this->verifyRedirectToURL('http://localhost/team');
 
         // Now check if we actually have this clarification.
-        self::assertSelectorExists('html:contains("problem C")');
+        self::assertSelectorExists('html:contains("C")');
         self::assertSelectorExists('html:contains("I don\'t understand this problem")');
     }
 }

webapp/tests/Unit/Entity/ClarificationTest.php

@@ -15,7 +15,7 @@ public function testSummary(): void
 'Lorem ipsum dolor sit amet, consetetur sadipscing elitr, sed diam nonumy eirmod
 tempor invidunt ut labore et dolore magna aliquyam erat, sed diam voluptua.';
         $clarification->setBody($text);
-        static::assertEquals('Lorem ipsum dolor sit amet, consetetur sadipscing elitr, sed diam nonumy eirmod …',
+        static::assertEquals('Lorem ipsum dolor sit amet, consetetur sadipscing elitr, sed diam nonumy eirmod tempor invidunt ut labore et dolore magn…',
             $clarification->getSummary());
     }
 
@@ -50,9 +50,10 @@ public function testMergeNewlines(): void
 sixth line,
 seventh line,
 eighth line,
+ninth line,
 and so on.';
         $clarification->setBody($text);
-        static::assertEquals('First line, second line, third line, fourth line, fifth line, sixth line, sevent…',
+        static::assertEquals('First line, second line, third line, fourth line, fifth line, sixth line, seventh line, eighth line, ninth line, and so …',
             $clarification->getSummary());
     }
 }