-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Description
README
[Does the README describe the project using the subheadings: Why?, What?, How? i.e. Why have you created this repo, what does it do, how does it do it?]
- Seems to repeat the brief we were given. This might cover why.
Possibly needs more detail to cover what? and how?.
User stories
[Does the project meet the user stories for that week?]
- Yes it does! (Screen reader not checked)
Learning outcomes
[Does it demonstrate the learning outcomes for that week?]
- Yes learing spikes seem to be covered - a lot of CSS learning and their CSS documnet looks good.
UI bugs
[Can you see any obvious bugs or areas to improve?]
button on the form could be made the same size as the input boxes
Instructions
[Does everything work as expected or were there missing instructions?]
File structure
[When you open the project in your editor, does the file structure make sense?]
- Yes structure looks good
Flow of control
[ Can you you follow the different paths the code might take?]
Naming
[Do variables and functions have clear and descriptive names?]
- Yes
Readability
[Do you understand the code?]
- Yes
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels