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

Advice generator site

#accessibility#fetch

@zubyCharles

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


The project was a really simple one though, took a couple of hours. I finally did get to use the browser fetch method in a real site though and enjoyed chaining multiple then methods.

Community feedback

Yuliya 230

@Ambrolla

Posted

Hi!

I've noticed some things, which will look better if you'll change them.

  1. In the quote, you forgot to add quotes.
  2. Also, I'd change the text color to plain white.

But you did a good job! I wanted to go with your method at first too, but then I went with asynch, await function, which I've heard is better.

Marked as helpful

1

@zubyCharles

Posted

@Ambrolla thanks for the feedback. Yeah I didn’t quite notice the absence of quotes, and I agree the text should be white for more contrast. I’m just hearing that async is better now but there shouldn’t be much difference as this is just a tiny fetch operation.

0

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