pabloanovillo
commented
4 years ago Criticas visuales, opinión personal:
- ¿Porque el input email tiene dos puntitos al final de la palabra? Debería sacarlos.
- La tipografía de los input debería verse un poco más cuando se completa con el email.
- El footer me parece que rompe con la armonía de la vista, al ser en color blanco, también me parece innecesario el scroll, debería estar dentro del viewport inicial, al estilo navbar.
Criticas a nivel código:
- Los layouts sos estructuras generales para vistas que comparten la misma estructura, no veo conveniente que el layouts del login se llame Main, debería llamarse login y compartirse para todas las vistas referidas, login, forgot, etc., o un nombre acorde, Main se utiliza comunmente para la vista principal, home.
- No entiendo la estructura de los componentes, hay "Atoms" que tiene un monton de componentes que no corresponden a la filosofía de atomic desing, después hay otra carpeta patterns que adentro hay una carpeta atoms y molecules y tiene archivos sueltos fuera de esas dos.
- La estructura de archivos de styles debería corresponder a la de los componentes, si vamos a seguir la ideología de Atomic Desing. El archivo donde se encuentran las variables genereales (ej: paleta de colores) yo le pondría de nombre variables, porque tenes un index y un zeepcommerce. Quedo un archivo colgado navbar.scss dentro de styles.
- ¿Se va utilizar sass o jss o ambos?.
Criticas de lógica:
- El link "get started" no realiza ninguna acción. (No se si correspondía darle la lógica en esta issues).
cristianvena18
closes #80
agregado además la configuración de next bien hecha, y agregado los estilos estandarizados
NOTA: correr un yarn install