NFT Preview HTML, CSS & FLEXBOX

Community feedback

• 

  Danilo Blas•6,300

  @Sdann26Posted over 2 years ago

  Alejandro, mucho gusto!

  Permiteme hacerte algunas correciones a tu trabajo:

  • Cambia el div con la clase container por un main ya que esta etiqueta tiene el uso de tener el contenido principal de la página.
  • Al div que tiene el texto el texto Equilibrium cambialo por un h1. Siempre una página debe tener mínimamente un h1. Aunque por lo general nunca se le suele poner a un componente en este caso como es un proyecto chiquito tiene sentido.
  • Siempre que uses una imagen ponle el atributo alt ya qe este ayuda a los lectores de pantallas que no pueden leer imagenes a saber de que se trata además que muestra un texto sobre que se trata la imagen si no llega a cargar por algún motivo.
  • Para poner estilos siempre usa clases o en menor caso las etiquetas nunca los id ya que estos tienen la finalidad de ser unicos y no son reutilizables haciendo que generes más lineas de código CSS.

  Por ahora son las recomendaciones que puedo darte. Espero que te haya sido de ayuda!

  Marked as helpful

  1
• 

  Alejandro Hernández Rosa•180

  @AlexuvaPosted over 2 years ago

  Gracias por las observaciones Danilo! Tendré en cuenta todas las observaciones en el siguiente challenge

  Un saludo! Mil gracias :D

  1