Hola @angyecastillo y Berenice
He estado revisando su proyecto y tengo las siguientes recomendaciones en las que pueden mejorar su proyecto:
El proyecto no tiene test, por lo tanto no han tenido un enfoque orientado a test, y su código no es testeable.
Al momento de correcto el test de validator nos muestra un warning, les sugiero que lo ocupen con todos sus archivos html antes de sus entregas para que puedan corregir dichos errores.
Les recomiendo empezar a utilizar HTML semántico, por ejemplo utilizar etiquetas como: <header></header> , <nav></nav>, <section></section>, <articule></articule>, <footer></footer>, <aside></aside>.
Importante que ya empecemos a abstraer y reusar código en todos los archivos; por ejemplo en el archivo de css tienen la propiedades que son repetidas en muchos elementos diferentes, pudiendo poner dicha propiedad en una clase, y aplicar esa clase en los elementos que lo requieran.
Recuerden que trabajar colaborativo o no, debemos de tener buenas practicas has escribir el mensaje de un commit y que las demás personas (o tu yo del futuro) entiendan que se cambió en dicho commit. les dejo este link que les puede ayudar.
No utilizaron ninguna configuración ni siguen ninguna guía de estilos (linter), en Laboratoria les recomendamos seguir la guía de airbnb.
Muchas felicidades por sus alcance en este primer proyecto. Las invito a que sigan trabajando en el proyecto y dejarlo aún mucho mejor. 👍
Hola @angyecastillo y Berenice He estado revisando su proyecto y tengo las siguientes recomendaciones en las que pueden mejorar su proyecto:
<header></header> , <nav></nav>, <section></section>, <articule></articule>, <footer></footer>, <aside></aside>
.Muchas felicidades por sus alcance en este primer proyecto. Las invito a que sigan trabajando en el proyecto y dejarlo aún mucho mejor. 👍