Skip to content
Submitted 6 months ago

Responsive FAQ accordion

shaunso•30
@shaunso
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

What are you most proud of, and what would you do differently next time?

Most proud of completing the challenge completely. It would be easy to just leave out one aspect of the challenge, but tenacity is imperative.

Next time, I would ensure that I code for tablets even if the challenge doesn't call for it

What challenges did you encounter, and how did you overcome them?

The keyboard navigation.

Looked at the FAQ of actual companies to see how they handled this to discover that

This wasn't intuitive to me at first, but I do understand now.

What specific areas of your project would you like help with?

Looking for help on how to refactor the JavaScript in my 'app.js' file. It feels rough & dirty to me and I feel like I could simplify the code.

Community feedback

  • Manav•390
    @manav-sharma69Posted 6 months ago

    Hii! I looked at your JS and HTML code.

    Most of your JS code wouldn't be needed if you use the <details> element. Also, using this tag would make your markup more semantic and accessible.

    Here's the MDN link: <details>: The Details disclosure element

    Hope this helps 🙏

    Marked as helpful
    1
    shaunso•30
    @shaunsoPosted 6 months ago

    Thank you so much @manav-sharma69! I had not come across the <details> element before somehow; you've been a tremendous help.

    1
    Manav•390
    @manav-sharma69Posted 6 months ago

    Happy to help! 😊

    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