advice generator , consuming an API with javascript

Community feedback

• 

  Danilo Blas•6,300

  @Sdann26Posted over 2 years ago

  Hola José!

  Felicitaciones por acabar este reto :D, en lo personal te recomendaría que siempre que hagas algún elemento en la web que sea posiblemente clickeable le agregues el atributo cursor: pointer para que sea más intuitivo que es un botón y no solo una imagen.

  Por cierto el error de accesibilidad se generá porque por defecto cuando creas un botón sueles colocarle texto dentro pero en el caso de que denas colocar imagenes o SVG en la mayoría de casos, lo que harás es agregarle el atributo aria-label el cual servirá a la hora de usar lectores de pantalla saber que hacer tal botón o link. Por ejemplo podrías poner el botón <button onclick="obtenerDatos()" aria-label="Change to advice">.

  Con eso solucionarias ese problema que tienes y bueno hay otras cosas que puedes mejorar como a la card darle un max-width para que no se agrande más de lo que debería, ya que aunque no lo pidan en el reto ya en los trabajos hasta se suele pedir que hagas que el diseño se mantenga aunque este con un tamaño de 1920px por ejemplo.

  Espero que mis comentarios puedan haber servido de ayuda, buena suerte :D!

  Marked as helpful

  0

  ---

  

  José Rodríguez•90

  @josergzPosted over 2 years ago

  @Sdann26 Desconocia por completo esa etiqueta para mejorar la accesibilidad, muchas gracias por tu retroalimentacion, tomaré en cuenta cada una de tus recomendaciones para seguir mejorando, gracias:D

  1