Feedback Project Social Network

[AmaliaRiveraC]

1. Test: Al correr eslint nos marca 1 error justo dentro de data.spec.js, cuidado en este archivo están mezclando la sintáxis de es5 y es6 a la hora de declarar sus funciones anónimas dentro de describe, [es5 function(){}, es6 () => {}].
2. DRY: Aún no llegan al nivel 3 que pretende que haya abstracción y rehuso de estilos a través de clases, por ponerles un ejemplo ustedes están declarando la propiedad font-family en más de una ocasión con el mismo valor 'Montserrat Alternates', pudiendo tener una clase que se encargue de esto y utilizarla en las distintas etiquetas html donde la necesiten.
3. HTML/correctitud/validación: el archivo de index.html al ingresarlo como código a https://validator.w3.org/ arroja 1 error, pero el archivo view1.html arroja 5 errores, no olviden ingresar su código en esta url antes de sus entregas.
4. HTML/Semántica/nomenclatura: cuidado con la divitis en el archivo index.html, el archivo view1.html utiliza etiquetas como section, la idea es llegar en todos los archivos html a esto! El objeto de configuración de firebase sólo lo deberían de declarar una vez, no se necesita en los dos archivos.

Buenas prácticas: nombrar clases y ids con guión medio (ej. label-publicacion). Cuidar indentación en el archivo main.css. Revisen los lineamientos de markdown, tiene su propia sintáxis para subir links o imágenes. Cuidado con los bugs al editar los post y al tratar de crear cuenta.

*Las felicito por su trabajo. Refleja compromiso y atención al detalle. Mejorando estos puntos mínimos tendrán un trabajo de alta calidad para mostrar en sus protafolios.

Cualquier duda que tengan no duden en acercarse a preguntar. Que el éxito llegue pronto !!! Un abrazo grande.

[AmaliaRiveraC]

Pronto estarán recibiendo su issue sobre UX!