-
Notifications
You must be signed in to change notification settings - Fork 231
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
Random guess button #70
base: main
Are you sure you want to change the base?
Conversation
not dictionary-based, sadly - performance problems
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Final comment: Won't this just populate the line with pseudo-random characters? I think it would be more useful to populate the line with a random word from the wordlist - perhaps only enabled in settings as a handicap option.
Yeah it's more of a discussion than a feature. Dictionary is a performance problem.
|
Why not just call |
This is what I mean: https://github.com/Oreolek/hello-wordl/pull/1/files |
use onKey and randomTarget for random word
Thank you, that's brilliant! |
Love this feature! @lynn perhaps you could make this toggle-able under settings if you don't want it available by default? |
Press the button to make a totally random guess!
(Not dictionary-based, sadly - performance problems. Dictionary is too big to filter for a specific word length.)