Skip to content

[Phoebe&Fran's Project] - code review by Lucien  #4

@Luciensday

Description

@Luciensday

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?]

README to be added

User stories

[Does the project meet the user stories for that week?]

  • Team information are completed but Phoebe and Fran's info should also be added to them

  • Website is responsive and can be view comfortably across all device

  • Click to navigate to different sections of your webpage

  • Contact you to enquire about working with you

  • Visit your website at a publicly accessible domain name Domaine name to be setup
    Additionally, as a visually impaired user, I want to:

  • Navigate your website using keyboard controls. tested the site can be navigagted with tab key

  • Hear my screen reader describe the content on your website.

Learning outcomes

[Does it demonstrate the learning outcomes for that week?]

UI bugs

[Can you see any obvious bugs or areas to improve?]

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?]

Flow of control

[ Can you you follow the different paths the code might take?]

Naming

[Do variables and functions have clear and descriptive names?]

Readability

[Do you understand the code?]

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions