Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Cocial Proof Section HTML CSS

Dust98• 130

@Dust1100010

Desktop design screenshot for the Social proof section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


how could i improve my good practice skills? ✌

Community feedback

Shawn Lee• 560

@OGShawnLee

Posted

¿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

@Dust1100010

Posted

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

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord