Skip to content

FTBC6: Ray #50

Open
R4Y815 wants to merge 4 commits intorocketacademy:mainfrom
R4Y815:master
Open

FTBC6: Ray #50
R4Y815 wants to merge 4 commits intorocketacademy:mainfrom
R4Y815:master

Conversation

@R4Y815
Copy link

@R4Y815 R4Y815 commented Jan 26, 2022

Please fill out the survey before submitting the pull request. Thanks!

🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀

How many hours did you spend on this assignment?
52 hours.

Please fill in one error and/or error message you received while working on this assignment.
('null') reading at xxx.Add.EventListener....

What part of the assignment did you spend the most time on?

  • the logic code for reading each of the winning hands particularly 3-of-a-kind / 1-pair .

Comfort Level (1-5):
4

Completeness Level (1-5):
3

What did you think of this deliverable?
This is a good assignment to test and promote understanding of CSS, html, DOM and JS because it tests how well the coder understands how to link between these different areas, while not being overly complex.

Is there anything in this code that you feel pleased about?
Despite the insufficient time, it is still a playable game in the basic form, and I like that i would get to test it out on my own phone

…d cards un-limitedly 3) CSS workiing for phone screens and 1920x1080, using position:relative on cardHolder and cardImg. (cannot set cardImg position to absolute)
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.

1 participant