QR Code Component

Community feedback

• 

  Danilo Blas•6,300

  @Sdann26Posted over 2 years ago

  Martinsito, buen proyecto te ha quedado bastante bien y muy bien usado las propiedades CSS!

  Quería agregar a que este <div class="card"> lo cambies por <main class="card"> o que uses un <main> para englobar a tu card. El motivo del main es que es una etiqueta muy importante en la semántica de la web ya que engloba el contenido principal del body. Esto lo mencionó para eliminar esos errores en tu reporte y puedas generar uno sin ningún error.

  Además podrías agregarle una ligera sombra con la propiedad box-shadow: 0 12px 12px -2px rgba(0, 0, 0, 0.2);

  En si todo el proyecto esta bien pero lo que quiero recalcar es el uso de <main> para que agregues a tus demás proyectos para que no tengas ningún error.

  Good Coding :D!

  Marked as helpful

  1

  ---

  

  Martinsito Brito Diaz•130

  @MartinsitoBritoDiazPosted over 2 years ago

  @Sdann26 Gracias por tu aporte, tomare en cuenta lo del uso del main y la ligera sombra.

  1
• 

  Diego Lucero•0

  @dieluceroPosted almost 3 years ago

  Very well done!!

  1