Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive advice mini-app with fetch

@maikCyphlock

Desktop design screenshot for the Advice generator app coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
  • API
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Do you have any corrections for my proyect.?

what do you think about my css practices ?

what do you think about my javascript practices ?

Community feedback

Danilo Blas 6,300

@Sdann26

Posted

Hola Maikol!

Te recomiendo cambiar el div que tiene el .attribution por un footer cosa que no te saltará ese problema de accesibilidad que tienes. El otro problema no se exactamente porque se da 3: pero generá un nuevo reporte por siacaso.

Respecto al diseño creo que te quedo bastante bien y funciona bien para diferentes navegadores.

Buena suerte!!

Marked as helpful

1

@maikCyphlock

Posted

@Sdann26 gracias danilo por tomarte tu tiempo para revisarlo, ok tomare en cuenta lo del footer para la próxima. feliz de que te haya gustado el diseño.

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord