Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feedback #1

Open
karlanavam opened this issue Jun 27, 2018 · 0 comments
Open

Feedback #1

karlanavam opened this issue Jun 27, 2018 · 0 comments

Comments

@karlanavam
Copy link

Si bien el proyecto cifra y descifra correctamente, debes tener en cuenta lo que el cliente pide, en este caso se solicitaba que el usuario pudiera ingresar la cantidad de desplazamiento (offset), nunca debemos olvidarnos de esos detalles y nosotros decidir deliberadamente quitar requisitos.
No hay visión de TDD, tienes al archivo index.js haciendo todo sin separar las responsabilidades entre logica y DOM.
Los identificadores en javascript son descriptivos en su mayoría, pero hay algunos que no me dicen mucho como por ejemplo: numAsq, textNum, debes trabajar ser lo más descriptiva posible.
Te recomiendo seguir estudiando y analizando el código, ahora que ya tienes todo el contenido más interiorizado te animo a re-hacer el proyecto, estoy segura hay muchas cosas que visualizarás, las que en un primer momento no fueron evidentes.

En HTML te recomeindo revisar la indentación y usar etiquetas semánticas.

En CSS muy bien por hacer los estilos en una hoja externa, ahora sólo te invito a que revises qué propiedades se repiten para que las clasifiques y las puedas reutilizar en el HTML sin duplicar estilos en CSS.

En UX me encantá que hayas pensado en el usuario para proponer tu solución, pero te invito a que los argumentos sean más profundos y justificados 🙂 Recuerda que un proceso rico en UX está destinado al éxito!!! 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant