search

DianaCarrillo / cdmx-2018-06-bc-core-am-cipher

https://dianacarrillo.github.io/cdmx-2018-06-bc-core-am-cipher/src/
1 stars 0 forks source link

Feedback Project Cipher #1

Open AmaliaRiveraC opened 6 years ago

AmaliaRiveraC commented 6 years ago
  1. Sobre el nivel de investigación: concientes de que es un punto un tanto subjetivo, he podido notar que utilizas 'addEventListener' lo cual me denota nivel uno, aunque noto que se incluye el cdn de bootstrap y no son utilizadas las clases, te sugeriría indagar un poco más en temas de css y buscar profundizar en el proceso de UX research.
  2. Documentar en el README info básica sobre requerimientos, instalación y uso, y una vez entradas en UX documentar la investigación de mercado si es que hubo, y el proceso en general.
  3. En la línea 31 del index.html el atributo id del

    debería de estar entre comillas. Si tenemos errores en el html no corren los test.

  4. Correctitud index.html: Agregar atributo lang en la estiqueta <html lang="es">. Agregar diagonal de cierra en las etiquetas , falta cierre en linea 7 y 9. Puedes apoyarte en los links de validación (https://validator.w3.org/) para ver y corregir tus errores antes de entregar!
  5. Utilizar etiquetas semánticas del HTML como header, nav, main, section, footer, aside.
  6. Dentro del addEventListener se manejan las funciones anónimas o solo se pone el nombre de la función y la función se declara en otra línea de código. 
    cipherButton.addEventListener("click", getData)
const getData = () => {
bloque de código
}

    *Como buena práctica nombrar id y clases con guión medio (ej. cipher-friends)

Buen proceso y mucha actitud! Sigue así de comprometida y llegarás muy lejos!

DianaCarrillo commented 6 years ago

Amalia, muchas gracias por comentar el issue. Gracias por mencionar que me ha faltado estudiar más sobre bootstrap y css en general, pero estoy segura que con la práctica me irá mejor. De los trabajos de mis compañeras aprendí cómo hacer la documentación del proceso y ahora es más claro qué debo hacer. Los errores de sintaxis los dejé al último, pero sé que debo acostumbrarme a escribir lo mejor posible desde el inicio y hacer una revisión final de ello. Gracias por los comentarios detallados y por las porras!

AmaliaRiveraC commented 6 years ago

Se me olvidaba Diana, busca que tus commits sean más descriptivos de lo que hacen. Saludos!