Social Proof Section | HTML CSS

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted almost 2 years ago

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

  • To properly center your content to your page, you will want to add the following to your Body element (this method uses CSS Grid):

  body {
      min-height: 100vh;
      display: grid;
      place-content: center;
  }
  

  More Info:📚

  Centering in CSS

  • To not only improve your HTML code but to identify the main content of you page, you will want to wrap your entire component inside the main element.

  More Info:📚

  MDN Main Element

  • The “star icons” serve no purpose other than to** be decorative;** They add no value. Their Alt Tag should left blank and have an aria-hidden=“true” to hide them from assistive technology.

  More Info:📚

  https://www.w3schools.com/html/html_images_picture.asp

  • • The profile images are not decorative. Their Alt Tags should not be blank. It should state the following; “Headshot of -person’s full name-“

  More Info:📚

  https://www.w3schools.com/html/html_images_picture.asp

  • To improve the semantics of your component, you want 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:

  ARTICLE

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

  Happy Coding!🎄🎁

  Marked as helpful

  0

  ---

  

  novadior1k•100

  @Novadior1kPosted almost 2 years ago

  @vcarames Wow! Thank you so much for your help, I will make sure to look out and utilize those tips, again thank you!!

  1

  

  バレンタイン 😈•64,170

  @VCaramesPosted almost 2 years ago

  @Novadior1k

  Glad I could help!

  Keep it up!

  0