Skip to content
OverviewSolutions
10
Comments
5
Chris-Ann C.
@cdc2021

All comments

  • Chris-Ann C.•300
    @cdc2021

    @oluwapelumi

    Great job on this challenge!

    Suggestions:

    • Have a read of your report as you have some accessiblility issues, maybe try adding a label for "email" which needs to match your input id rather than 'action'.
    • Ensure you are using the headings in chronological order and applying to sections.
    • The second page is very close to the first page so I would add some padding or margin to separate the elements.
    • The testimonial cards sits behind the access card, maybe have a look into your positioning.

    Hope this is useful.

    Happy coding :)

  • Chris-Ann C.•300
    @cdc2021

    Great job on this project! Looks and feels good. One thing your burger menu disappears when scrolling so maybe think about the colour scheme as it dissolves into the white background.

    Haappy coding !

  • Chris-Ann C.•300
    @cdc2021

    Great work on this challenge, very responsive!

    Minor details, the contents within your header are quite close so maybe adding some padidng betweeen to space out the h2 and p tag. Also the opacity of the rely class should match the power class rather than the p tag.

    Hope this is useful.

    Happy coding :)

  • Chris-Ann C.•300
    @cdc2021

    Great, thank you :)

  • Chris-Ann C.•300
    @cdc2021

    Hi Alejandro,

    I would remove the background img from the HTML file and style in CSS using background-image: url(./); remove repeat - background-repeat: no repeat In order to position the images use tranform: tranlate (%, %) This should solve your background image issues and so make styling of the container easier.

    Hope this is useful. Chris-Ann