Social Proof Section Challenge using Flexbox and SASS

Community feedback

• 

  P

  tediko•6,680

  @tedikoPosted over 3 years ago

  Hello, Rizwan Mustafa! 👋

  Good effort on this challenge! Your solution responds well. Here's my few suggestions:

  • You repeat your HTML code with stars images which is unnecessary. Find the way to use only one star image for a .ratingStars. You can do it easily by using background-repeat approach in your css.
  • .customerName should be a heading <h1-h6> element.
  • .customerStatus and .feedback should be paragraph <p> or <span> elements instead of div.

  Good luck with that, have fun coding! 💪

  1

  ---

  

  Rizwan Mustafa•155

  @rizwanmustafaPosted over 3 years ago

  @tediko Thanks for the suggestions!

  0
• 

  Shreya Choure•45

  @shreyachourePosted over 3 years ago

  Hey, Great work! If you use line-height: 1.7 (try values around 1.5 - 1.9)for the paragraph content after headline it would look even more elegant!

  1

  ---

  

  Rizwan Mustafa•155

  @rizwanmustafaPosted over 3 years ago

  @shreyachoure Thanks, I missed that part!

  0