adalab / modulo-1-evaluacion-final-cerosales Goto Github PK

Muy buen trabajo @CeRosales!!!

Aquí tienes unos consejos para mejorar tu ejercicio:

• Migra tu proyecto al Adalab Starter Kit.
• Revisa el indentación de tu código.
• HTML: Todos los img deben tener el atributo alt:

  modulo-1-evaluacion-final-CeRosales/index.html

  Line 29 in 63cfe2f

  <a href="https://adalab.es/"><img src="./images/hamburger.png" class="imagen-hamburguesa" /></a>

• HTML: El texto de Disfruta creando espacios debe estar en un h1:

  modulo-1-evaluacion-final-CeRosales/index.html

  Line 43 in 63cfe2f

  <h2 class="h2-espacios">Disfruta creando espacios</h2>

• HTML: No es necesario englobar el texto "Comprar ahora" del botón en una etiqueta p:

  modulo-1-evaluacion-final-CeRosales/index.html

  Line 101 in 63cfe2f

  <button class="module-3__button"><p class="module-3__text-button">Comprar ahora</p></button>

• HTML > FOOTER: No es necesario englobar el texto de los li dentro de etiquetas de p, modifícalo cada vez que sea necesario en

  modulo-1-evaluacion-final-CeRosales/index.html

  Lines 125 to 138 in 63cfe2f

  	<a class="footer__1" href="https://adalab.es/" target="_blank"> <p class="footer__tittle">Tienda</p></a>
  	<a class="footer__1" href="https://adalab.es/" target="_blank"> <p class="footer__text" >Productos</p></a>
  	<a class="footer__1" href="https://adalab.es/" target="_blank"><p class="footer__text">Cajoneras </p></a>
  	<a class="footer__1" href="https://adalab.es/" target="_blank"> <p class="footer__text" >Para toda la casa</p></a>
  	<a class="footer__1" href="https://adalab.es/" target="_blank"> <p class="footer__text">Habitaciones </p></a>

• HTML > FOOTER: Incluye las etiquetas nav para el menú del footer. No es correcto que esté englobado eb un articulo el listado de enlaces:

  modulo-1-evaluacion-final-CeRosales/index.html

  Line 121 in 63cfe2f

  <article class="footer__tienda">

• HTML > FOOTER: No es necesario tener un footer y luego una section, cuando son el mismo footer:

  modulo-1-evaluacion-final-CeRosales/index.html

  Lines 118 to 119 in 63cfe2f

  	<footer>
  	<section class="footer__section">

• CSS > ESTILOS REPETIDOS: Hay estilos repetidos en las media queries que no son necesarios, es mejor quitarlos. .module-3__section--2 { background-image: url(../images/creando-espacios.jpg) }, .module-3__section--2 { background-size: cover }
• CSS: El hero debe medir 800px de alto cuando el tamaño de la ventana es 800px.
• CSS: El hero debe medir 1000px de alto cuando el tamaño de la ventana es 1000px.
• CSS > TRANSICIONES: Anímate a crear alguna transición CSS.

Es posible que cuando corrijas uno de estos puntos, otros se corrijan solos, ya que unos dependen de otros.

Y recuerda algo muy importante:

• Escribe tu código en inglés.
• Escribe todos los ficheros y carpetas en inglés.
• Los commits también tienen que ser en inglés, empezando por un verbo en infinitivo.

Ánimo y si tienes cualquier duda avísame!!!

Con cada mejora que hagas, márcala como completada en este issue y su súbela a este repo.

Este issue ha sido creado automáticamente por ello puede tener algún error, si hay algo que no te encaja escribe por slack al coordinador de Adalab.

Para corregir el ejercicio buscamos determinadas palabras en el código para identificar las secciones. Por ejemplo buscamos Creando para buscar el título de la página. Si has escrito mal alguna palabra en el código es posible que te aparezcan aquí falsos errores.

Por favor no cierres este issue, ya lo haremos nosotros.