vaniushar / cdmx-2018-06-bc-core-am-cipher Goto Github PK

Muy buena separación de responsabilidades entre lógica (cipher.js) y la manipulación del DOM (index.js), el producto funciona perfectamente, eso si, no te animaste a completar los test lo que indica que jamás desarrollaste analizando los casos de prueba. Espero en el siguiente proyecto si lleves a cabo la metodología TDD.

En la parte de HTML te recomiendo investigar más obre las etiquetas semánticas y a incorporarlas en tu desarollo a la hora de plantear una lógica en la estructura. También a que propongas una solución Responsive para que el usuario pueda acceder desde su celular a la plataforma 🙂

En CSS, lo mejor que se puede hacer es crear los estilos desde una hoja externa. Esto te va a ayudar al momento de querer hacer una corrección. A veces los archivos van a tener miles de líneas de código y para llegar a corregir 1 elemento va a ser muy tedioso. Si todos los planteas desde una hoja externa, puedes crear una estructura donde sepas dónde va cada uno. También aprovechar otras clases que tengas para repetir código lo menos posible; DRY 😉

En la aprte de UX, me encanta que hayas pensando en el usuario, pero sin duda, se pueden dar más argumentos del porqué y cómo de la solución para la necesidad del usuario. Tú puedes 🙂

Reloaded:

• Separar variables como globales

• Completar el Readme con UX research

• Agregar más comentarios a mi código.

• Separar todas las cuestiones de estilo de html a style.css