AmaliaRiveraC
commented
6 years ago - Documentación: Al readme le hace falta la información sobre instalación y uso.
- Investigación: Busquemos llevar un poco más lejos nuestra curiosidad aplicando el Object.values(), lo veo comentado en su código, anímense a usarlo.
- Nomenclatura: Existen algunas variables que nombre sigue siendo críptico o inconsistente. HTML:
- Estilo (htmlhint): Corramos en consola el comando npm run htmlhint, si no lo tienen instalado en sus computadoras pueden dar npm install -g htmlhint.
- Validación/Correctitud: Antes de sus entregas metan su códgio HTML en este link https://validator.w3.org. Podrán identificar los errores de sintáxis y corregirlos para subir su nivel. Source Code Managment
- Git: Las he dejado en el nivel 3 que contempla pull request y merge, pero en el nivel 2 se espera convenciones de commits, este debería de ser un acuerdo inicial con el equipo de trabajo, para entender cómo vamos a escribir los commits, de tal manera que sean accesibles para un tercero.
- Github: Hubo un taller con Alexandra de uso de issues y milestones, se esperaba el manejo de estas funcionalidades, pongan atención en este punto, no es un tema complejo y las ayudará a su planificación. UX:
- Entrevistas: Documentemos estos procesos.
- Alineación: Cuidado con la alineación de los botones, y el tamaño que disponemos para las cajas de texto, sobre todo en la versión responsive la alineación es un área de oportunidad.
- Contraste: Hay cierta intuición de lo que implica el contraste, aunque los colores de botones y de letra de botones o los textos dentro de la tabla, deberíamos buscar que desde el color y el tamaño el texto diga, qué datos son los más importantes en la tabla de estudiantes, por ejemplo.
- Felicidades por el esfuerzo, sigan trabajando duro y pongan atención en los puntos mencionados, mientras vayan aplicando el feedback también irán mejorando en sus procesos. No duden en escribirme para lo que necesiten.
RaquelGlez