Feedback técnico

[Gabx04]

Recomendaciones de mejora 👍

README: Muy bien que documentes lo encontrado sobre tu usuario y el producto. También puedes añadir como te organizaste durante estos 2 sprints, imágenes de la evolución de tu producto, las tecnologías que utilizaste.

UX: Muy bien que describas a los usuarios de tu producto y los objetivos de tu producto.

HTML: Recuerda usar etiquetas semánticas al construir tu estructura. No olvides validar tu html, te recomendamos revisar el enlace: https://validator.w3.org/nu/?doc=https%3A%2F%2Fanaeyguillen.github.io%2Flim-2018-05-bc-core-am-cipher%2Fsrc%2F Para solucionar el error en el link a google fonts revisa este link o también puedes importar las fuentes

CSS: Recuerda indentar correctamente tu código. En la línea 40 estás usando la pseudo-clase :focus, super bien que la estes probando, como observación, la forma correcta de usarla es con dos puntos adelante la pseudo-clase (Ej. :link, :visited, :focus, etc).

Estilos para input que tiene focus input:focus { /* Aquí van tus estilos */ } Estilos para input que tiene la clase focus input.focus { /* Aquí van tus estilos */ } JS:

• cipher.js: Te animamos a terminar el cifrado para minúsculas. Revisa la formula de descifrado para offsets grandes, al probar con números de 3 dígitos en adelante la formula arroja un resultado incorrecto. Al momento de desarrollar alguna función te sugerimos que trates de probar con varios números y hasta con valores o argumentos “extremos” o “raros”. Ten en cuenta que tu usuario puede probar con cualquier número al momento de usar tu página.

• index.js: Te recomendamos hacer uso de const para las variables btnencode, btnlimpiar, etc, ya que su valor no cambia. Recuerda declarar tus variables con nombres descriptivos, en inglés y usando camelCase. Por último, declaras las variables offset, ingMensaje dos veces en tu código, podrías optimizar el código y hacer la declaración de ambas variables al inicio y fuera de los addEventListener.

Tests: Cumples con la cobertura requerida para el proyecto, en el caso de no realizar el hackher edition con la función createCipherWithOffset puedes quitar el test para que no muestre un error de fallido.

Sigue así buen trabajo 👍

[AnaeyGuillen]

Gracias Gaby, tendré en cuenta tus recomendaciones para ajustar mis próximos proyectos.