Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Sunnside landing page

IRVINE MESA• 1,855

@DrMESAZIM

Desktop design screenshot for the Sunnyside agency landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


I put in great effort to match up to the challenge. All feedback is welcome

Community feedback

P
Fluffy Kas• 7,735

@FluffyKas

Posted

Hey,

Your solution looks great and I can tell you put a lot of effort into this! The image-placement in some sections look a bit strange apart from the widths defined in the style guide but I haven't done the challenge so I can't really say how it could be done better. Looks neat enough anyway! There are some bits where this could be improved though:

  1. I don't see any font-family imported, have you forgot this perhaps?

  2. I see some empty selectors in your css file, you could remove these.

  3. For your navigation you should use a <nav>.

  4. You should add aria-labels to your social icons! Title isn't the best solution for this.

Everything else looks great, you even thought of adding some really nice alt texts to you images. Good job again ^^

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord