Skip to content

Instantly share code, notes, and snippets.

@the-oem
Last active March 23, 2017 15:23
Show Gist options
  • Save the-oem/a556c22601f1c5eefb70ce4e7a751b6e to your computer and use it in GitHub Desktop.
Save the-oem/a556c22601f1c5eefb70ce4e7a751b6e to your computer and use it in GitHub Desktop.
Turing Module 1 - Number Guesser Known Issues

Known Issues

  1. There is no error checking to validate that min range value is less than max range value.

New requirements suggestions

  1. Guess button should be disabled when the input field is empty, because guessing with no value doesn't really make sense. This wasn't in the requirements, but in a real world scenario I would have asked the product owner about this.

Things I would like to have completed

  1. More refactoring. Plenty of this left to do, and I am not following the DRY rule as much as I would like.
  2. Clean up unused CSS rules.
  3. Make the min and max input fields flash once with the pink color when their values change automatically by -10 and +10, respectively to help inform the user that there was a change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment