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 agency landing page with flexbox

@PedroS2001

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


I appreciate all kinds of opinions. Mainly comments about the navbar, since it's the first time I've done it without bootstrap. Thanks

Community feedback

Rio Cantre 9,690

@RioCantre

Posted

Hello there! Awesome work with this challenge. Looking at your solution, I think you should know the following as well…

  • Import the attribution style in CSS file and remove the style tag
  • Clean the whitespaces in the code
  • Include description with the alt in image tags but it's optional

Other than that...

  • Proper usage of semantic tags
  • The HTML structure is organized and readable
  • Well done in using Sass for styling with separate files for specific target elements. The compositions are done properly
  • The navbar is done accordingly, it transition into a hamburger menu once in smaller viewport
  • The project is responsive and utilized the details based on the original design
  • The hover state of the buttons and links of the project is done properly as well

Above all, the project is well implemented. Keep up the good work!

Marked as helpful

0

@PedroS2001

Posted

@RioCantre Thank you so much Rio! Your comment is very helpful. Those little details make the difference.

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