Skip to content
Submitted over 2 years ago

HTML SCSS Jquery Ajax

Saleh•390
@SalehAbuhussein
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

it's my first time Doing Challenge with Jquery nothing fancy I just fetched the Data, I think in vanilla JS it's done with fetch if I am not wrong. any feedback is appreciated.

Community feedback

  • Victor•1,035
    @CodeVeePosted over 2 years ago

    @Honko-o Hi Saleh, Great job on this. A few things I would say are

    • Good use of destructuring. Adds Clarity

    • JQuery was a bit of overkill. Still good though.

    • p can be an h1 tag for accessibility.

    • a should be a button. Makes for better Markup

    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