Skip to content
Submitted over 3 years ago

Simple Accordion Card with JavaScript

Andrew•260
@andy-devs
A solution to the FAQ accordion card challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

My first attempt to make an accordion. Looking forward to hear your advice :)

Code

Please log in to post a comment

Log in

Community feedback

  • Dharmik_487•1,740
    @Dharmik48

    Hey👋,

    Your solution is really good! But.. I found a few suggestions:

    • Try to use semantic html tags like main, section, header, etc. more as it is a good practice and good for SEO.
    • Also, I think you should remove the margin-top: 10rem; from 886px media-query as because of it there is a vertical scrollbar.

    Keep it Up👍

    Marked as helpful

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