Skip to content

Initial Room and Statistics Page UI#130

Merged
git-yash merged 24 commits into
mainfrom
feat/statistics-display
Apr 6, 2026
Merged

Initial Room and Statistics Page UI#130
git-yash merged 24 commits into
mainfrom
feat/statistics-display

Conversation

@CalvinT05
Copy link
Copy Markdown
Contributor

@CalvinT05 CalvinT05 commented Mar 30, 2026

Changelog

Initial Statistics Page UI

  • Updated to React 19 for charts (shad/recharts)
  • Added shad/recharts
  • Participants stats
    • Bar Charts
    • Tabulated
  • Score meter
    • Radial Bar Chart
    • Score breakdown on hover
  • Submission stats
  • Last submitted code text box

Initial Room Page UI

  • Participant list
  • Admin icon

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
bsg Ready Ready Preview, Comment Mar 30, 2026 8:57pm

Request Review

@CalvinT05 CalvinT05 marked this pull request as ready for review March 30, 2026 22:04
Copy link
Copy Markdown
Contributor

@Eskan0r Eskan0r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works fine on my end and looks good well done, statistics page needs to have better spacing between its components you can make subtabs if you want perhaps but it should not need to scroll horizontally to see everything on the default size when attached. you can set it to look like this when the size of the panel is big enough, also please remove the log for the screen size

@Eskan0r
Copy link
Copy Markdown
Contributor

Eskan0r commented Apr 2, 2026

anyone else reviewing make sure there are no errors when you run it, there are murmurs of issues and ts needs to be chill

@git-yash git-yash merged commit 8793326 into main Apr 6, 2026
3 checks passed
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.

4 participants