Social Proof component with basic CSS/HTML

Community feedback

• 

  P

  Emmilie Estabillo•5,540

  @emestabilloPosted about 4 years ago

  Hi Benjamin, good job completing this challenge! Here are some thoughts:

  • The background for desktop is also being used for mobile widths

  • I would look into multiple backgrounds so you don't have to manage the background images with pseudoelements and z-index

  • At the 900px breakpoint, the component is almost touching the gutter. It is also not occupying the entire viewport height.

  • .review is missing display: flex for desktop

  • Lastly, you should address the issues in the report with missing alt text, since it is an html standard and the images will be inaccessible.

  Hope this helps :-)

  0

  ---

  

  Benjamin Sabardu•90

  @bsabarduPosted about 4 years ago

  Hi, thanks a lot for your feedback, i'll gonna have a look and do some updates.

  Benjamin

  0