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 text field to BoardSelector #31

Merged
merged 2 commits into from
Sep 30, 2023

Conversation

dylan-chong
Copy link
Contributor

@dylan-chong dylan-chong commented Sep 16, 2023

to allow quick pasting in of flop cards from other tools

Considerations when defocusing the input:

  • the loading of cards into the model is done by clearing the board and through the existing toggleCard function, so that handles the 5 card limitation
  • text is parsed and loaded into the model. the text is then regenerated from the model to ensure correct formatting
  • if you type non-letter symbols (other than comma or space), they are stripped, to allow easy pasting of data from other programs
  • if you type invalid cards, they are simply ignored and removed from the input
  • cards are case insensitive. as per point 2, they are correctly capitalized automatically

Toggling cards via the buttons of course updates the input

image

to allow quick pasting in of flop cards from other tools
@dylan-chong dylan-chong mentioned this pull request Sep 28, 2023
@b-inary b-inary merged commit dfbd4db into b-inary:main Sep 30, 2023
@b-inary
Copy link
Owner

b-inary commented Sep 30, 2023

Merged, thank you for your contribution!

b-inary added a commit to b-inary/wasm-postflop that referenced this pull request Sep 30, 2023
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.

None yet

2 participants