Testimonials grid section / solution

Community feedback

• 

  Heli0s•670

  @zeerobitPosted about 2 years ago

  Well done completing your first grid challenge. I got a few pointers:

  • your h1 and h2 are actually paragraphs and should be inside of blockquote.
  • Replace your section tags with figure tags and use figcaption to wrap the user info, click this link to read more about these elements.
  • none of the images has alt and that's bad for accessibility.
  • The cards need a box-shadow

  Happy coding !!!

  0
• 

  David•8,000

  @DavidMorgadePosted about 2 years ago

  Buenas Pato, se nota que has mejorado bastante desde el ultimo challenge! te quedo bastante bien el grid, enorabuena

  Como consejo lo único que te diría es que intentaras centrar tu main usando flex en el body, puedes hacerlo facilmente así:

  body {
    font-family: 'Barlow Semi Condensed', sans-serif;
    margin: 0;
    background-color: #ecf2f8;
    min-height: 100vh;
    display: flex;
    justify-content: center;
    align-items: center;
  }
  

  El width del body no hace falta que lo fijes al 100% ya que es así por defecto.

  Y ya si quieres dejarlo perfecto para desktop, podrías ponerle margin: 0 1rem al <article> y así no tendrías margin ni arriba ni abajo y dejarias espacios a los lados!

  Espero que mi feedback te sea de ayuda, si tienes cualquier duda pregunta lo que sea!

  0