[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?]
Good ToC
Nice headings and sub-headings
User stories
[Does the project meet the user stories for that week?]
Tabbing through website misses lots of sections.
Responsiveness looks good.
Nice use of repeatable styling, maybe need more css primitives to cut down on code?
Nice grid structure used for team members.
Learning outcomes
[Does it demonstrate the learning outcomes for that week?]
UI bugs
[Can you see any obvious bugs or areas to improve?]
Nav-bar isn't a list (
) so screen reader struggles and isn't clickable. Also needs tab indexes
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?]
yep.
Flow of control
[ Can you you follow the different paths the code might take?]
yep.
Naming
[Do variables and functions have clear and descriptive names?]
yep.
Readability
[Do you understand the code?]
Good readability. nice commenting on code throughout.
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?]
Good ToC Nice headings and sub-headings
User stories
[Does the project meet the user stories for that week?]
Tabbing through website misses lots of sections. Responsiveness looks good. Nice use of repeatable styling, maybe need more css primitives to cut down on code? Nice grid structure used for team members.
Learning outcomes
[Does it demonstrate the learning outcomes for that week?]
UI bugs
[Can you see any obvious bugs or areas to improve?]
Nav-bar isn't a list (
) so screen reader struggles and isn't clickable. Also needs tab indexes
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?]
yep.
Flow of control
[ Can you you follow the different paths the code might take?]
yep.
Naming
[Do variables and functions have clear and descriptive names?]
yep.
Readability
[Do you understand the code?]
Good readability. nice commenting on code throughout.