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

sunnyside solution in HTML and CSS

AiltonG 110

@AiltonG

Desktop design screenshot for the Agency landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


All feedback is walcome! thanks you in advance

Community feedback

Alexandra 550

@Alexandra2888

Posted

Hi! Congrats for finishing challenge! Your solution is very close to the design! Here are some things i think you can improve:

  • implement semantic HTML,
  • make CSS reset: *, *::after, *::before {margin:0; padding:0: box-sizing: border-box;};
  • try to use either classes or ids,
  • choose better names for classes or implement BEM naming convention. It will be easier in bigger projects and when you will work with other developers,
  • convert px into relative units (em/rem). Px are usually used when coding for magazines/newspapers and you need fix units. Happy coding!

Marked as helpful

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