página de qr con flex

Community feedback

• 

  P

  Luis Vera•190

  @LuisVera1Posted 7 months ago

  Hola @Rod-RodriguezF-28

  ¡Tu solución se ve muy bien!, aqui te dejo algunos consejos para mejorar la semántica.

  --- Html

  • Usa la etiqueta <main> para envolver todo el contenido en lugar de la etiqueta <div>
  • Usa primero la etiqueta <h1> y después <h2> y asi, hasta el <h6>, y en ese orden.
  • Los <br> que tienes dentro del titulo y del párrafo no son necesarios, estos se ajustan segun el tamaño del contenedor.

  ---Css

  • en tu .card puedes usar la propiedad padding para darle espacio entre el contenido y el borde y asi no necesitas el .container-card
  • A la imagen te recomiendo ponerle un width de 100%, este tomará el 100% del elemento padre, si este reduce su tamaño la imagen tambien se reducirá, lo que ayuda al diseño responsivo.

  Revisa este link: QR-base

  Espero esto te sea útil, y si lo es no olvides marcarlo.

  Muy buen trabajo, ¡sigue asi!

  Marked as helpful

  0

  ---

  

  Rodrigo Rodriguez•10

  @Rod-RodriguezF-28Posted 7 months ago

  @LuisVera1 Muchas gracias por tu feedback, me servio muchisimo, saludos.

  0
• 

  P

  Daniel Clement•80

  @DanielClement37Posted 7 months ago

  the component is a bit bigger and if the img is sized a bit bigger it will shrink down the padding around it.

  Marked as helpful

  0

  ---

  

  Rodrigo Rodriguez•10

  @Rod-RodriguezF-28Posted 7 months ago

  @DanielClement37 Muchas gracias por tu feedback, me sirvio mucho, saludos.

  0