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

created flashing number + status of flash + result checker #42

Merged
merged 5 commits into from
Feb 12, 2025

Conversation

rik102
Copy link
Collaborator

@rik102 rik102 commented Feb 11, 2025

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

Copy link
Owner

@ATOMiNATiON ATOMiNATiON left a 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. 👍

src/pages/flash.astro Show resolved Hide resolved
src/pages/flash.astro Show resolved Hide resolved
src/pages/flash.astro Show resolved Hide resolved
src/pages/flash.astro Show resolved Hide resolved
src/pages/flash.astro Outdated Show resolved Hide resolved
src/pages/flash.astro Show resolved Hide resolved
src/pages/flash.astro Outdated Show resolved Hide resolved
* 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');
Copy link
Collaborator

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

src/pages/add.astro Show resolved Hide resolved
src/pages/add.astro Outdated Show resolved Hide resolved
src/pages/add.astro Outdated Show resolved Hide resolved
src/pages/add.astro Outdated Show resolved Hide resolved
src/pages/add.astro Outdated Show resolved Hide resolved
Copy link
Collaborator

@JacksonWKennedy JacksonWKennedy left a 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

Copy link
Collaborator Author

@rik102 rik102 left a 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
Copy link
Collaborator

@s5bug s5bug left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@ATOMiNATiON ATOMiNATiON merged commit ce1ce85 into ATOMiNATiON:main Feb 12, 2025
1 check passed
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.

5 participants