Skip to content

Conversation

brinky
Copy link
Contributor

@brinky brinky commented Sep 3, 2025

When submission had no changes the diff is empty and there was no way in the interface to switch to edit files.
Even worse is the case of only added files. There the files are not even available in a previous submission.

If that would be a relevant use case, the interface could be improved somehow to show more information than an edit and download button. But that might mean to merge submission_source and submission_diff.

Currently buttons for all changed files are shown at once. Only the
buttons of the selected file should be shown.
@brinky brinky force-pushed the submission_diff_unchanged branch 2 times, most recently from 10fc358 to 9d3352f Compare September 3, 2025 10:53
@brinky brinky force-pushed the submission_diff_unchanged branch from 9d3352f to 88946df Compare September 3, 2025 10:58
@brinky brinky marked this pull request as ready for review September 3, 2025 10:58
@tuupke tuupke merged commit ff7f00c into DOMjudge:wfbaku Sep 3, 2025
2 checks passed
eldering pushed a commit that referenced this pull request Sep 3, 2025
meisterT pushed a commit to meisterT/domjudge that referenced this pull request Sep 6, 2025
meisterT pushed a commit to meisterT/domjudge that referenced this pull request Sep 7, 2025
github-merge-queue bot pushed a commit that referenced this pull request Sep 7, 2025
(cherry picked from commit 3957fcd)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants