Skip to content

FTBC 5-1 David Video Poker#28

Open
daves77 wants to merge 13 commits intorocketacademy:mainfrom
daves77:master
Open

FTBC 5-1 David Video Poker#28
daves77 wants to merge 13 commits intorocketacademy:mainfrom
daves77:master

Conversation

@daves77
Copy link

@daves77 daves77 commented Oct 19, 2021

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

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

How many hours did you spend on this assignment?
approx 15-20hrs

Please fill in one error and/or error message you received while working on this assignment.
Lots of guess and check work with CSS

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

Comfort Level (1-5):
5
Completeness Level (1-5):
5
What did you think of this deliverable?
Wonderful lead up to the current project, really helps put everything we learnt from the past 2 weeks together

Is there anything in this code that you feel pleased about?
The design came out pretty nicely

Copy link

@michelle-mok michelle-mok left a comment

Choose a reason for hiding this comment

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

good work organising your code into files!
code is well organised into functions and easy to read :)
consider adding more comments that explain your code so that it is easier for the reader to follow your code

</div>
</div>
</div>
</main>

Choose a reason for hiding this comment

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

elements should be placed on the board using DOM in this project

}
}
return false;
};

Choose a reason for hiding this comment

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

consider adding more comments that explain your code to the reader

@@ -0,0 +1,25 @@
/*global document */

Choose a reason for hiding this comment

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

consider adding comments to explain what this code does

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.

2 participants