From b3d6969cb0965946ea4fd067c82f229d4e77e815 Mon Sep 17 00:00:00 2001
From: Michael Vasseur <14887731+vmcj@users.noreply.github.com>
Date: Sun, 28 Sep 2025 13:03:09 +0200
Subject: [PATCH 1/3] Simplify template

---
 webapp/templates/jury/submission_source.html.twig | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/webapp/templates/jury/submission_source.html.twig b/webapp/templates/jury/submission_source.html.twig
index 2178173b03..7e5ed75522 100644
--- a/webapp/templates/jury/submission_source.html.twig
+++ b/webapp/templates/jury/submission_source.html.twig
@@ -41,7 +41,9 @@
         <p><a href="#origdiff">Go to diff to original submission</a></p>
     {%- endif %}
 
-    {% if not oldSubmission %}
+    {%- if oldSubmission %}
+        {%- include 'jury/partials/submission_diff.html.twig' with {oldSubmission: oldSubmission, oldFiles: oldFiles, oldFileStats: oldFileStats} %}
+    {% else %}
         <ul class="nav nav-tabs source-tab-nav">
             {%- for file in files %}
                 <li class="nav-item">
@@ -74,10 +76,6 @@
         </div>
     {% endif %}
 
-    {%- if oldSubmission is not null %}
-        {%- include 'jury/partials/submission_diff.html.twig' with {oldSubmission: oldSubmission, oldFiles: oldFiles, oldFileStats: oldFileStats} %}
-    {%- endif %}
-
     {%- if originalSubmission is not null %}
         <h2 id="origdiff" class="mt-3">
             Diff to original submission

From 49efb21bd2a9fea0de2a92b505f3cf5535389d4a Mon Sep 17 00:00:00 2001
From: Michael Vasseur <14887731+vmcj@users.noreply.github.com>
Date: Sun, 28 Sep 2025 16:11:53 +0200
Subject: [PATCH 2/3] Only display one sourcecode tab at a time

---
 .../jury/partials/submission_diff.html.twig   | 23 +++++++++++--------
 1 file changed, 13 insertions(+), 10 deletions(-)

diff --git a/webapp/templates/jury/partials/submission_diff.html.twig b/webapp/templates/jury/partials/submission_diff.html.twig
index c991494b33..00d4a5d679 100644
--- a/webapp/templates/jury/partials/submission_diff.html.twig
+++ b/webapp/templates/jury/partials/submission_diff.html.twig
@@ -27,22 +27,23 @@
     {%- endfor %}
     {%- for file in oldFileStats.unchangedfiles %}
         <li class="nav-item">
-            <a class="nav-link {% if loop.first %}active{% endif %}" data-bs-toggle="tab"
-               href="#diff-{{ file.submitfileid }}" role="tab">{{ file.filename }}</a>
+            <a class="nav-link {% if loop.first and oldFileStats.changedfiles == [] %}active{% endif %}"
+               data-bs-toggle="tab" href="#diff-{{ file.submitfileid }}" role="tab">{{ file.filename }}</a>
         </li>
     {%- endfor %}
     {%- for file in oldFileStats.addedfiles %}
         <li class="nav-item">
-            <a class="nav-link {% if loop.first %}active{% endif %}" data-bs-toggle="tab"
-               href="#diff-{{ file.submitfileid }}" role="tab">{{ file.filename }}</a>
+            <a class="nav-link {% if loop.first and oldFileStats.changedfiles == [] and oldFileStats.addedfiles == [] %}active{% endif %}"
+               data-bs-toggle="tab" href="#diff-{{ file.submitfileid }}" role="tab">{{ file.filename }}</a>
         </li>
     {%- endfor %}
 
 </ul>
 <div class="tab-content source-tab">
     {%- for filePair in oldFileStats.changedfiles %}
-        <div class="tab-pane fade {% if loop.first %}show active{% endif %}" id="diff-{{ filePair.1.submitfileid }}"
-             role="tabpanel">
+        <div class="tab-pane fade
+                    {% if loop.first %}show active{% endif %}"
+             id="diff-{{ filePair.1.submitfileid }}" role="tabpanel">
             {% set diff_id = "diff" ~  filePair.1.submitfileid %}
             <div class="mb-1">
                 <a class="btn btn-secondary btn-sm"
@@ -67,8 +68,9 @@
         </div>
     {%- endfor %}
     {%- for file in oldFileStats.unchangedfiles %}
-        <div class="tab-pane fade {% if loop.first %}show active{% endif %}" id="diff-{{ file.submitfileid }}"
-             role="tabpanel">
+        <div class="tab-pane fade
+                    {% if loop.first and (oldFileStats.changedfiles == []) %}show active{% endif %}"
+             id="diff-{{ file.submitfileid }}" role="tabpanel">
             <div class="mb-1">
                 <a class="btn btn-secondary btn-sm"
                    href="{{ path('jury_submission_source', {submission: submission.submitid, fetch: file.rank}) }}">
@@ -84,8 +86,9 @@
         </div>
     {%- endfor %}
     {%- for file in oldFileStats.addedfiles %}
-        <div class="tab-pane fade {% if loop.first %}show active{% endif %}" id="diff-{{ file.submitfileid }}"
-             role="tabpanel">
+        <div class="tab-pane fade
+                    {% if loop.first and (oldFileStats.changedfiles == []) and (oldFileStats.addedfiles == []) %}show active{% endif %}"
+             id="diff-{{ file.submitfileid }}" role="tabpanel">
             <div class="mb-1">
                 <a class="btn btn-secondary btn-sm"
                    href="{{ path('jury_submission_source', {submission: submission.submitid, fetch: file.rank}) }}">

From 86725a75b1ce07a99d514fa1a85ff96c5d33a46b Mon Sep 17 00:00:00 2001
From: Michael Vasseur <14887731+vmcj@users.noreply.github.com>
Date: Sun, 28 Sep 2025 13:33:22 +0200
Subject: [PATCH 3/3] Display all files for a submission

---
 webapp/templates/jury/partials/submission_diff.html.twig | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/webapp/templates/jury/partials/submission_diff.html.twig b/webapp/templates/jury/partials/submission_diff.html.twig
index 00d4a5d679..55953736f9 100644
--- a/webapp/templates/jury/partials/submission_diff.html.twig
+++ b/webapp/templates/jury/partials/submission_diff.html.twig
@@ -83,6 +83,7 @@
                     </a>
                 {% endif %}
             </div>
+            {{ file.sourcecode | codeEditor(file.rank, submission.language.editorLanguage) }}
         </div>
     {%- endfor %}
     {%- for file in oldFileStats.addedfiles %}
@@ -101,6 +102,7 @@
                     </a>
                 {% endif %}
             </div>
+            {{ file.sourcecode | codeEditor(file.rank, submission.language.editorLanguage) }}
         </div>
     {%- endfor %}
 </div>