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 App

Chris 130

@Cortele

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


First time working with APIs; I enjoyed this project, not comfortable with async JS, going to continue practising with it. I had a hard time accessing the element within the object to update my innerHTML, but figured it out with the help of some people on the frontendmentor Discord.

Community feedback

Aimal Khan 2,260

@Aimal-125

Posted

Bro in your css code, increase the height of body element to 120 or 150vh by using media query with max-height: 400px; So that it looks good on small heighted screens as mine (samsung j3). Otherwise your solution is looking perfect!

Marked as helpful

1
Chris 130

@Cortele

Posted

Thank you to w3dg and Cereal on Discord for helping me out!

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