SunnySide agency landing page with flexbox

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted over 2 years ago

  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

  ---

  

  Pedro Señeriz•30

  @PedroS2001Posted over 2 years ago

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

  0