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

Letter distribution chart #77

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

JoshuaFox
Copy link
Contributor

@JoshuaFox JoshuaFox commented Sep 3, 2021

This creates a chart as follows:

  • This is the first added feature, in contrast to a bug fix. So, it deserves more caution.
  • As requested, this does not alter board size. Also, in a typical browser, the right area was just wood-texture background, so this fits well.
  • The server adds the letter distributions to the game data it sends.
  • In rendering the board, this distribution received in gameData is used to build the chart on the right.
  • The CSS is adjusted minimally to get a stable appearance. If there are design considerations, please let me know or tweak this.

image

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