Skip to content
Submitted almost 3 years ago

Advice Generator App using CSS Flexbox

Ayoife•300
@Ayoife
A solution to the Advice generator app challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Wow!, my first Junior challenge! , I'm hoping you'll leave an honest review of this solution. And I also added some animations....Were they good?, were they too much?, Let me know in your feedback!. And also, I'm facing a problem with increasing the height of the container as the text increases, I'll be glad if you could help me with that bug. :)

Code

Please log in to post a comment

Log in

Community feedback

  • Rimsha•240
    @rimshub

    Hey @Ayoife good solution!! Animations are looking good and smooth.

    Some points i noticed while testing:

    1. The pattern divider overlaps with the dice-icon (button) when text of advice increases. I think you should set some height of paragraph as well, not only width in advice class. OR see if you can set the pattern divider image position to not change when text length increases.

    2. Also, its better idea to go with one animation. It looks too much when type of animation changes with every advice. Otherwise animations really look nice!!

    I hope this helps. Keep going✨

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