SASS, CSS flexbox, Parcel, Mobile first, async functions

Community feedback

• 

  Danilo Blas•6,300

  @Sdann26Posted over 2 years ago

  Esteban, muy buen trabajo, el diseño y la interactividad esta muy buena.

  Arreglemos los errores de tu reporte, siempre debes arreglarlos ya que te ayudan a ser mejor desarrolador frontend. Entonces, veamos:

  • Tiene un problema por no usar un <main> dentro de tu body. Tienes dos opciones colocar la card dentro del body y al <div class="attribution"> lo cambio por <footer class="attribution"> o tenemos la opción de agregar ambos dentro del body. Te recomiendo la primera opción ya que el attribution lo has puesto como si fuera pie de página.
  • Al <button class=button role=button id=buttonNewAdvice> no es necesario agregarle el atributo role.
  • Hay un error de Buttons must have discernible text es porque el botón no tiene texto y es normal porque tiene el logo del dadito, te recomiendo ponerle el atributo aria-label="Button to change the advice", esto es para que al usar lectores de pantalla al llegar al botón lea para que sirve este botón no solo sea algo que esta ahí y no entenderlo. Para más info busca accesibilidad en la web, explican los atributos ARIA.
  • Tienes diferentes opciones para eliminar el ultimo error que es no ponerle un h1 ya que siempre debe haber un h1 por pagina, puedes usar el ADVICE con el número como titulo y luego ya le das estilos.

  Solo eso, espero que mis comentarios te ayuden a mejorary puedes aplicarlo en los demás proyectos para ser mejor desarrollador frontend, suerte.

  0