Social proof section built with HTML and CSS

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted almost 2 years ago

  Hey there! 👋 Here are some suggestions to help improve your code:

  • Your HTML should have a main element that will assist in identifying the main content of your site. In this, case the main element would be wrapping the entire component.

  More Info:📚

  MDN Main Element

  • The profile images Alt Tags need to be improved. It should state the following; “Headshot of -person’s full name-“

  More Info:📚

  Headshot Alt Text

  • For the testimonials, it is best to to wrap each individual testimonial component in a figure element, the individuals information should be wrapped in a figcaption element and lastly, the testimonial itself should be wrapped in a blockquote element.

  Code:

  <figure>
     <figcaption></figcaption>
     <blockquote></blockquote>
  </figure>
  

  More Info:📚

  MDN Figure Element

  If you have any questions or need further clarification, feel free to reach out to me.

  Happy Coding!🎄🎁

  Marked as helpful

  1

  ---

  

  Adán Estévez•320

  @rafdidactPosted almost 2 years ago

  Hey! I really appreciate your suggestions. Seems like I gotta work on my use of semantics.

  I have a question, should I use a main tag even though my HTML code doesn't follow a conventional webpage structure? Because I didn't wrapped all my content inside of a main element because I thought this was kind of pointless if I wasn't going to use any header and/or footer elements as well.

  1

  

  バレンタイン 😈•64,170

  @VCaramesPosted almost 2 years ago

  @rafdidact

  Glad I could help!☃️

  For this challenge, it is meant to be part of a much larger site, as the name of the challenge implies.

  I didn’t mention it, since the report stated it, but there is a footer; the attribution. (If you haven’t gotten a chance to read the report, I highly suggest you do, as it includes valuable information)

  While there may not be a header there still a there is a still a single section, which ultimately is the main content of your site.

  Pretty much everything you build will have a main element, regardless of how small a component it is.

  Hopefully, that clears things up.

  Keep it up!

  1