Querida Marian,
Felicitaciones. Cumplís a la perfección todas las consignas y se nota que acá hay mucho tiempo y esfuerzo invertido. Desde el primer momento frente a tu web sabemos que estamos frente a un trabajo muy bien hecho, pero viendo en profundidad tu código, se nota realmente lo bien que está. Tengo poco que agregar; se nota tu esfuerzo y se celebra tu aprendizaje.
Tu sitio cumple muy bien los requsitos de accesibilidad. Utilizas a la perfeccion las etiquetas semanticas, por lo que un lector de pantalla puede orientarse facilmente en tu web. Los colores y contrastes son en general adecuados y utilizas bien las etiquetas aria y el atributo alt. Siempre hay un foco visible y es facil utilizar la web con el teclado. Nada que agregar aqui, solo destacar el enorme trabajo en todo sentido, se nota mucha investigacion y esfuerzo para hacer tu web accesible; creo que nunca vi un sitio comercial que las usara tan bien como vos.
Tus filtros funcionan a la perfeccion. No solo eso, sino que la reutilizacion en responsive es perfecta. Nada que comentar aqui: se nota que hiciste un trabajo enorme y que pusiste mucho esfuerzo en que quedaran perfectos.
Tu carrito funciona muy bien, esta muy bien maquetado y cumple todos los requerimientos solicitados. Me gusta como resolviste los productos que se agregan o quitan: es distinto a como lo hizo el modelo de Ada, y eso habla de tu esfuerzo por pensarlo sola y solucionarlo con lo que sabes.
Todo perfecto en terminos de funcionalidades. Destaco lo prolijo del maquetado.
Se agradecen los detalles a cada instante, que revelan el cuidado por tu TP. El cambio en el boton comprar para indicarle al usuario que una accion esta ocurriendo es maravilloso: mejoraste la web modelo, y se nota que estabas pensando desde la perspectiva del usuario que podria no entender qué pasaba. Excelente.
Tu HTML esta perfecto. Excelente la indentacion, claro, completo, con las etiquetas semanticas adecuadas. Tu CSS tambien esta muy prolijo y bien hecho, reutilizas bien los estilos y los nombres de clases son claros y descriptivos. Se nota muchisimo el avance desde el portfolio hasta hoy.
Tengo que destacar especialmente la calidad del JS. Todos los comentarios que dejas son utiles para el lector, quiza pecas de excesiva pero en este punto de tu aprendizaje me parece la decision acertada. El orden es perfecto. Ni un solo console log olvidado.
Con respecto al comentario que dejas sobre declarar las variables dentro de las funciones, apruebo la decision aqui, ya que es cierto que es un codigo largo y es util saber donde se utiliza cada cosa. En terminos de perfomance, no es lo ideal repetir declaraciones de funciones (esas tendrian que estar declaradas de manera general), pero cuando son variables que solo se usan en esa funcion, esta perfecto.
Tenes muchos y muy buenos commits, y ni hablar de la calidad de tu README. Este es un trabajo del que estar muy orgullosa.
En resumen, hiciste un enorme trabajo, casi ningun problema en el producto entregado y con una enorme atencion al detalle y la calidad. Solo lamento no tener mucho que ofrecerte para que mejores tu trabajo, pero poco podemos hacer los docentes frente a TPs de esta calidad. Felicitaciones de todo corazon: sin dudas Marian estas en el lugar correcto y tenes un enorme futuro como desarrolladora front end.
Con respecto a los restantes factores de evaluación:
✅ Respeta la consigna. ✅ Estructura correcta de documento HTML. ✅ Respeta el diseño dado. ✅ Respeta el funcionamiento. ✅ Responsive funciona correctamente. ✅ Buena estructura de proyecto. ✅ Código bien indentado. ✅ Comentarios que permiten mejorar la legibilidad del código. ✅ Uso correcto de etiquetas semánticas. ✅ Buenos nombres de clases. ✅ Buenos nombres de funciones y variables. ✅ Reutilización de estilos. ✅ Funciones pequeñas. ✅ Commits con mensajes adecuados. ✅ Cumple con las condiciones de accesibilidad avanzada.
NOTA FINAL: 10