Skip to content
Submitted about 2 years ago

React, flexbox, axios

Matias Mass•280
@MatiasMass
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Advice generator app
  • HTML
  • CSS
  • JS
  • API
2junior

Design comparison

SolutionDesign

Solution retrospective

In this challenge I had to change advices for jokes because it didn't change the advices when I used axios. But the essence is the same.

Community feedback

  • GermanHeim•20
    @GermanHeimPosted about 2 years ago

    Hey! I liked the jokes twist :)

    Your container has an overflow: scroll; property that looks somewhat weird on desktop. If you change it to hidden, it looks way cleaner, in my opinion. You can see a preview of what I am saying in the following imgur link! Not assigning it a value should work too.

    Congrats on the app!

    1
    Matias Mass•280
    @MatiasMassPosted about 2 years ago

    @GermanHeim yes, you're right! I had to put in the p tag, I just fixed it. Thank you!

    1
    GermanHeim•20
    @GermanHeimPosted about 2 years ago

    @MatiasMass You're welcome!

    0

Please log in to post a comment

Log in with GitHub

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