Skip to content
OverviewSolutions
22
Comments
151

Favour

@Nadine-Green2,160 points

Learning more about this journey

I’m currently learning...

Javascript, SCSS, SQL, PHP

Latest solutions

View more solutions

Latest comments

  • Favour•2,160
    @Nadine-Green

    HEY YISHAK!

    I noticed that the screenshot generated does not match the website, you will need to generate a new one, just wanted to inform you.

    HOPE I COULD BE OF HELP :)

    HAPPY CODING!

  • Favour•2,160
    @Nadine-Green

    HEY FRANCI!

    I really like your design however I noticed that none of the active states were included, just thought I would inform you.

    HOPE I COULD BE OF HELP :)

    HAPPY CODING!

    Marked as helpful
  • Favour•2,160
    @Nadine-Green

    HEY BOHDAN!

    I noticed that a mobile design was not included in your solution, to ensure a good user experience, it is good practice to have a mobile design as well as a desktop design, you will need to address this.

    Instead of using a div for the <div class="attribution">, you should opt for a more semantic element like footer which will remove the error on your accesibilty report.

    HOPE I COULD BE OF HELP :)

    HAPPY CODING!

  • Favour•2,160
    @Nadine-Green

    HEY THERE!

    You will need to set a max-width for the desktop design of your elements because, if you notice, they keep expanding, adding this property will prevent them from doing that

    HOPE I COULD BE OF HELP :)

    HAPPY CODING!

  • Favour•2,160
    @Nadine-Green

    HEY ALEX!

    I believe for the change, the underline is supposed to be removed when the cursor hovers on it.

    HOPE I COULD BE OF HELP :)

    HAPPY CODING!

    Marked as helpful
  • Favour•2,160
    @Nadine-Green

    HEY NIKITA!

    Congratulations on completing your first challenge

    I really like your design

    However, I think you forgot to include an alt for your image, which is why you have a warning for it

    HOPE I COULD BE OF HELP :)

    HAPPY CODING!

View more comments