Skip to content
Submitted over 2 years ago

Flex box, mobile first design,

Jimmy D.•160
@Jd536
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

Hey Guys!

I hope you are enjoying the week, coding.

Submitting this right here and hope to hear your feedback

Cheers.☕

Community feedback

  • Abdul•8,540
    @SamadeenPosted over 2 years ago

    Hey!! Cheers 🥂 on completing this challenge.. .

    Here are my suggestions..

    • You should use <main class="card__content"> instead of <div class="card__content">.

    This should fix most of your accessibility issues

    . Regardless you did amazing... hope you find this useful... Happy coding!!!

    Marked as helpful
    0
    Jimmy D.•160
    @Jd536Posted over 2 years ago

    @Samadeen

    Thanks for pointing this out. Code updated.

    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