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

Social Proof

SheikBazithโ€ข 240

@SheikBazith

Desktop design screenshot for the Social proof section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Even though i finished code I am bit disappointed. Wish i coded it better. If any suggestions on how to approach better coding please comment.

Community feedback

@MelvinAguilar

Posted

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

@SheikBazith

Posted

@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

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