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

Initial box score work #13

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

iliana
Copy link
Contributor

@iliana iliana commented Jan 20, 2021

image

Draft until:

This should work correctly when games are in-progress, but doesn't auto-update. Not sure how I want to add that, because hitting the games/stats endpoint every 2 seconds is a bit more load than hitting the games/updates endpoint every 2 seconds.

Tested that this works on games where statsheets aren't loaded in Chronicler for a game (it just doesn't show the line score).

Future work: I'd like to make an expandable (hidden-by-default) part of the table that shows the full box score (per-player stats and also the game event list).

@iliana
Copy link
Contributor Author

iliana commented Jan 21, 2021

Mobile experience is pretty decent now.

FullSizeRender.mov

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.

1 participant