Skip to content
Submitted 12 months ago

FAQ accordion

Abraham-s-yakubu•100
@Abraham-s-yakubu
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...FAQ accordion
  • HTML
  • CSS
  • JS
1newbie

Design comparison

SolutionDesign

Solution retrospective

"The javascript was a bit of a challenge, not gonna lie, but I think I handled it well.

Community feedback

  • JimTK16•370
    @JimTK16Posted 12 months ago

    Hi Abraham, nice attempt at this challenge. My suggestion is to put the actions in a separate function (something like clickHandler) to keep your code easier to follow. Also, you might want to have clearer names for your variables. For example, if I quickly scan through your code, it would take me some time to figure out what is 'cls'. Hope it helps. Jim

    Marked as helpful
    1
    Abraham-s-yakubu•100
    @Abraham-s-yakubuPosted 12 months ago

    @JimTK16 thanks for the feedback I still new to JavaScript 😅

    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