Social Proof

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted 10 months ago

  Hello there 👋. Good job on completing the challenge !

  I have some suggestions about your code that might interest you.

  • You should change your <section> element to a landmark <main>.

  • Images should always have an alt attribute; otherwise, the filename will be read by screen readers. For icons, you can leave the alt attribute empty, but never omit it.

  • The person's photo should have an alt attribute with their name.

  • <span> is an inline element used for styling but is not a semantic tag. You should use paragraphs or headings to better describe your page's content.

  • I feel that the display: inline-block; and overflow: hidden; properties should not be on the body element. It's a common practice to limit the width of your current <section> with max-width: 1200px;.

  I hope you find it useful! 😄 Above all, the solution you submitted is great!

  Happy coding!

  Marked as helpful

  0

  ---

  

  SheikBazith•240

  @SheikBazithPosted 10 months ago

  @MelvinAguilar these are some of the best advices I got. I am preparing myself to code better and these will help me alot. Thanks

  0