Página de código QR con Scss y CSS Flexbox

Community feedback

• 

  Fernando Pérez•320

  @fermopPosted over 2 years ago

  Hola Matias! Excelente trabajo, tu página es responsiva para dispositivos móviles y en desktop.

  Veo que al momento de tener un móvil en horizontal en tu página no se puede deslizar hacia abajo para ver todo el contenido. Este problema me pasó a mí cuando realicé este proyecto y lo pude solucionar poniéndole un min-height con pixeles al body, ya que el container que tiene el card es más grande que el height de un celular en horizontal y esto no incrementa de tamaño al body debido a que tiene asignado el 100vh. (no estoy muy seguro de que sea por esto, pero si alguien puede corregirme se lo agradecería muchísimo jaja)

  También veo que en tu párrafo usas pixeles en su font-size. Te recomiendo usar rems, usualmente tienen un tamaño de 16px, usando rems permitirá que el tamaño de tus textos sea más responsivo en cada dispositivo, ya que no todas las personas tienen configurada el mismo tamaño de fuente. Este video explica muy bien los ems y rems: https://www.youtube.com/watch?v=_-aDOAMmDHI

  Por otra parte, veo que en el apartado de reportes te sugiere usar elementos semánticos tales como main y footer, estos no son de gran impacto para el usuario pero sí para leer el código y ubicar más fácil las secciones de nuestra página. Y por último veo que te sugieren utilizar un h1 ya que estás utilizando un h3. Recuerda que las páginas solamente pueden tener un h1 ya que este es el título de toda la página, después los subtítulos (h2), luego subsubtítulos (h3) y así hasta llegar a h6.

  De todas maneras hiciste un excelente trabajo, te ha quedado muy bien el proyecto!! saludos Matias :)

  Marked as helpful

  0