Skip to content
Submitted almost 3 years ago

Advice App

Tomiwa Orimoloye•1,000
@tomiwaorimoloye
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

Any feedback would be appreciated.

Community feedback

  • Rio Cantre•9,690
    @RioCantrePosted almost 3 years ago

    Hello there! Great work with this one. Regarding the solution you submitted, I think you should also take notes of the following…

    • Instead of div, alternatively wrap the whole content with main tag . Use semantic tags, for HTML structures, refer it with this one Semantics
    • Include description with the alt in image tags. Like this line <img class="divider" src="./images/pattern-divider-desktop.svg" alt="">

    Advice I got:

    Advice #158

    "Life can be a lot more interesting inside your head."

    Above all, The design looks good. Keep it up!

    Marked as helpful
    1

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