Fylo Landing Page using HTML and CSS

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted over 2 years ago

  Hello there! Nice job in completing this project. Regarding your solution, I would like to recommend the following for you…

  • Adjust the #testimonials .box-container .quotes properties into width: 6rem;, transform: transform: translate(-1rem, -5rem); and z-index: 1;
  • Include description with the alt in image tags. Like this line <img src="images/icon-phone.svg" alt="">
  • Adjust the padding in #testimonials .box rule set into padding: 3rem;
  • Increase the font size of #stay-productive .content h2 with font-size: 3.2rem;
  • Increase the padding in the header with padding: 5rem 8rem;
  • Wrap the sections with main tag. Use semantic tags, for HTML structures, refer it with this one Semantics

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

  0

  ---

  

  Ikenna•270

  @ikennaezefPosted over 2 years ago

  @RioCantre Thank you so much for the feedback. I've noted your suggestions and I appreciate them✨

  0
• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted over 2 years ago

  Hi Ikenna. Nice work so far.

  • the first heading element of your site should be an h1 and for best practice use only one h1 element.
  • alt values should be descriptive for the benefit of assistive technology users. Decorative images should have empty alt values.
  • A ul should be followed by a li element as a child. Check the footer social.

  0

  ---

  

  Ikenna•270

  @ikennaezefPosted over 2 years ago

  @ChamuMutezva Thank you so much for the feedback. I've noted your suggestions. I really appreciate them✨

  0
• 

  Kamasah-Dickson•5,590

  @Kamasah-DicksonPosted over 2 years ago

  Your solution looks great on mobile good job keep coding👍

  0

  ---

  

  Ikenna•270

  @ikennaezefPosted over 2 years ago

  @Kamasah-Dickson Thank you very much. I appreciate the comment

  0