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

Felicitaciones por el autoaprendizaje, se nota una investigación de tu parte para poder utilizar los eventos en el DOM. recuerda utilizar la metodología TDD, debemos empezar mirando los test, analizándolo para saber que necesitamos que nuestro código haga, por ejemplo nos pedía retornar un cierto resultado para un string y un offset dado y tú método jamás retorna solo utiliza el innerHTML, por eso no pasan los casos a pesar de funcionar correctamente. Separas la lógica de la manipulación del DOM, ese es un gran skill que te invito a seguir desarrollando, si quieres saber más puedes investigar sobre Patrones de diseño de software o MVC, Model - View - Controller.

Felicidades por plantear una solución responsive. Te recomiendo hacer la comprobación en la consola de Chrome para verificar se ve lo mejor posible para el usuario. Me parece que para dispositivos small se pierde un poco la funcionalidad y se prioriza la imagen. Fuera de eso, super!

En la parte del HTML te recomiendo usar etiquetas semánticas para proponer una lógica en la estructura.

Muy bien por hacer tus estilos en una hoja externa, ahora te recomiendo que los estilos que son generales y que se usan en todo momento, los clasifiques para usarlos una sola vez y si la intención es que se usen más de una, pues ver cómo se pueden clasificar para que los lleves a donde quieras, DRY 😉

En la parte de UX te felicito por plantear una solución pensando en el usuario, ahora te invito a que profundices en los argumentos 😄