All comments

• 

  Agustina• 45

  @aguscorvo

  Submitted about 3 years ago

  This is my second Frontend Mentor challenge. Mediaqueries were hard for this one.

  Any suggestions or possible improvements are welcome.

  Thanks!

  

  Social proof section with SASS

  2

  2

  0

  

  Hector Lopez• 195

  @hec-lopz

  Posted about 3 years ago

  Hi there!

  It looks great! Congrats on your solution. 🎉

  I'm not sure if you used the HTML <article> and <section> tags in the best way possible. It is my understanding that an <article> is a section of your pages that works by itself and can be replicated anywhere without loosing sense. A <section> represents a section separated from the others, but still requires its context to work. Given this, your .card-container and .rating-container should be sections themselves and not the title of each card, those titles could´ve been <h2> tags.

  No matter what, I really liked your solution! Hope you keep it coming and don´t hesitate to share your solutions. 😄

  1

  This comment has received 2 repliesShow