-
Notifications
You must be signed in to change notification settings - Fork 4
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
created flashing number + status of flash + result checker #42
created flashing number + status of flash + result checker #42
Conversation
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.
Nicely done implementing the flashing numbers, just need to add some adjustments to the flashing numbers and it should be good. 👍
* Fixed only numbers flashing on screen * Submission button created for users to click * Validation shows up only after users submits answers * Changed *Done* to *Answer*
- Changed placeholder to let user know what to input - Updated the # of numbers to flash to have 15 - Fixed speed min value to be 0.2, so it's doable
height: 100vh; | ||
margin: 0; | ||
font-family: Arial, sans-serif; | ||
background-image: url('/mathArenaBkg.jpg'); |
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.
I need to make a PR down the line moving all this stuff into a common stylesheet. Out of scope for this PR
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.
Basic struct in tact
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.
committed changes
- Resolve nitpick "a number" on all the field input - Resolve nitpick "sec" for speed
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.
Lgtm
shows if it's currently flashing or not and if it's done it'll show, and added the back button to go back to the prev and restart it, and it has a box where you can put in the answer and it'll alert if its right or wrong