Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add page with language stats #2647

Merged
merged 3 commits into from
Aug 16, 2024

Conversation

nickygerritsen
Copy link
Member

As discussed before.

Screenshots:
Screenshot 2024-08-11 at 15 21 40
Screenshot 2024-08-11 at 15 21 46

There is a button above the language graph to link to this page:
image

I made it two columns next to each other first, but expanding the teams will make a lot of teams two lines, so I didn't do that in the end.

Feedback on what to show or how to show it is welcome.

@vmcj
Copy link
Member

vmcj commented Aug 11, 2024

I would also be interested to see for which problems the language was used:
Lang:
1 team
Used for solving problems: [A][C]

@nickygerritsen
Copy link
Member Author

nickygerritsen commented Aug 11, 2024

Stealing some of @meisterT his awesome CSS skills to produce this:
image

@vmcj
Copy link
Member

vmcj commented Aug 11, 2024

Stealing some of @meisterT his awesome CSS skills to produce this: image

This will make that we need to fix the WCAG tests for jury (probably) as this has not enough contrast. As this is by design I'm fine with it.

@vmcj
Copy link
Member

vmcj commented Aug 15, 2024

The only thing missing for Astana is an easy export of this so we can directly share this when we start with EOC with whoever needs this? Is the plan to just save this page or should we also expose this via the API?

@meisterT
Copy link
Member

Keep it simple: Just print it and hand it over.

@nickygerritsen
Copy link
Member Author

After talking to @meisterT I improved it to this:
image

(don't mind the double in, I fixed that)

@nickygerritsen nickygerritsen added this pull request to the merge queue Aug 16, 2024
Merged via the queue into DOMjudge:main with commit 3e25ec9 Aug 16, 2024
23 checks passed
@nickygerritsen nickygerritsen deleted the language-stats-page branch August 16, 2024 09:10
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