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

QR code component

RGhstx 110

@rherndz

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


! Primera Practica Realizada ! 😄

🚀 ¡Cualquier comentario de ayuda o corrección de código es bienvenido! Estoy abierto a aprender y mejorar, así que no dudes en compartir tus conocimientos.

#HTML 🟠 #CSS 🔵

Community feedback

@MelvinAguilar

Posted

¡Hola! 👋. ¡Buen trabajo al completar el desafío! Tengo algunas sugerencias sobre su código que podrían interesarle.

HTML 📄:

  • Siempre evita saltar niveles de encabezado; Siempre comienza con <h1>, seguido de <h2>, y así sucesivamente hasta <h6> ( <h1>,<h2>,...,<h6>).
  • Debes usar un encabezado de nivel uno (h1) aunque este no sea un desafío completo de página. Tiene dos opciones:
  1. Cambiar su elemento <h3> por un <h1>
  2. Puede crear un elemento '<h1>' dentro de tu elemento 'main' que estará oculto visualmente pero visible y legible para lectores de pantalla. La clase "sr-only" oculta el contenido visualmente y aquí están los estilos para copiar. ej.: <h1 class="sr-only">QR Card Component</h1>
  • Dado que este componente implica escanear el código QR, la imagen no es una decoración. No debe usar la propiedad background-image para agregar la imagen del código QR. En su lugar, use la etiqueta <img> para agregar la imagen. Use la propiedad background-image solo para imágenes decorativas que no añadan ninguna información a la página.

CSS 🎨:

  • Para centrar un elemento verticalmente, debes usar una altura para su contenedor y utilize la propiedad align-items: center;. En este caso se recomienda usar "min-height: 100vh" para que ocupe el 100% del alto del viewport.
.container {
    min-height: 100vh;
    display: flex;
    /* flex-wrap: wrap; */
    /* column-gap: 30px; */ /* No existen otros elementos, esto es innecesario */
    /* row-gap: 40px; */ /* No existen otros elementos, esto es innecesario */
    justify-content: space-around; /* también se puede usar el valor "center" */
    /* height: auto; */ /*Utilize min-height en lugar de height*/
    /* margin: 40px 0; */
    /* padding: 50px; */
    align-items: center;   /* Agregue esto para centrarlo verticalmente */
    padding: 1rem;
}

¡Espero que te sea útil! 😄 Por encima de todo, ¡la solución que has presentado es genial!

Saludos!

Marked as helpful

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