Cocial Proof Section HTML CSS

Community feedback

• 

  Shawn Lee•560

  @OGShawnLeePosted over 2 years ago

  ¿Ey que pasa? Según tu perfil, eres de Perú así que te voy a escribir en Español colega xD.

  Bastante buena la solución! Se ve genial en todos los tamaños de pantalla. Solo que yo le pondría un poco mas de padding a los paneles con las estrellas en desktop, se ven un poco flaquitos en mi opinión.

  Sobre los testimonios, yo usaría un article como tag contenedor y usaría el nombre de las personas como el heading. En este caso sería un h3, pero para ello pondría un h2 para toda la section de los testimonios y lo haría sr-only para que solo lo vean screenreaders. Algo así:

  <section>
    <h2 class="sr-only">Testimonials</h2>
    <div>
        <article>
          <h3>/* nombre */</h3>
        </article>
   </div>
  <section/>
  

  Así usas HTML semántico y es mucho más fácil de leer y entender el código.

  Eso es todo mi feedback. Buena solución colega. Ten un excelente día! :D

  Marked as helpful

  0

  ---

  

  Dust98•130

  @Dust1100010Posted over 2 years ago

  @OGShawnLee Lo tendré en cuenta y gracias por el consejo ✌

  0