Skip to content

Conversation

@singto1597
Copy link

📝 Overview

This Pull Request introduces enhancements to the contest overview table to provide quick visual feedback on contestant scores

✨ Features Added

  • Score Highlighting: Implemented color coding for contestant scores in the overview table.
    • Scores that are 100% (Full Score) are highlighted in Green.
    • Scores that are above the passing threshold but not yet 100% are highlighted in Yellow (to indicate good progress but room for improvement).
    • Scores below the passing threshold are highlighted in Red.
  • Visibility: Ensures the score column is clearly visible and updates automatically.

❓ Why This Change?

The visual feedback from colored scores allows administrators and judges to quickly assess the performance of participants without having to navigate to detailed score pages. This significantly improves user experience and administrative efficiency.

⚠️ Note on Commits

  • Cleanup Commit: You might notice a commit labeled "Delete Folder" or similar in the history. This was only for reverting an accidental inclusion of unnecessary local script files and is not part of the intended feature changes.

🖼️ Visual Comparison

Before

image

After

image

@prandla
Copy link
Member

prandla commented Nov 1, 2025

  • Please skip the AI description next time. (There's numerous factual inaccuracies in it, I'm not even going to bother listing them since you obviously didn't write it yourself.)
  • You messed up the permissions on every file in the repo.
  • You added a git submodule which is unnecessary and unrelated to the rest of this PR.
  • The code itself is quite bad. Namely it both computes task scores incorrectly, and formats the scores incorrectly.
  • There is already an open PR which does this better: Add a Score counter to task overview #1476

PS:

Cleanup Commit: You might notice a commit labeled "Delete Folder" or similar in the history. This was only for reverting an accidental inclusion of unnecessary local script files and is not part of the intended feature changes.

Have you ever heard of git rebase?

@prandla prandla closed this Nov 1, 2025
@singto1597
Copy link
Author

Thank you for your detailed and honest feedback. I sincerely apologize for the numerous issues with this PR.
I want to clarify that the description was not entirely generated by an AI. I originally drafted the text in my native language and then used a translation tool to convert it to English, as I am not very proficient in English. I understand this process led to factual inaccuracies, and I take full responsibility. I will immediately rewrite the description myself to be accurate and specific.
I have acknowledged all the technical errors and will address them as next project.
Thank you again for your guidance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants